Skip to content

branch: 채팅 설정 api 연동#63

Merged
whitecity01 merged 24 commits intodevelopfrom
feature/setting
Nov 16, 2025
Merged

branch: 채팅 설정 api 연동#63
whitecity01 merged 24 commits intodevelopfrom
feature/setting

Conversation

@whitecity01
Copy link
Member

  • 채팅 설정 get, put 요청 연동하였습니다.
  • csrf 토큰 로직 삭제되어서 해당 기능을 제거했습니다.

@github-actions
Copy link

github-actions bot commented Nov 15, 2025

Visit the preview URL for this PR (updated for commit 5010c4d):

https://web-deploy-9d41a--pr63-feature-setting-8jmgrngv.web.app

(expires Sun, 23 Nov 2025 13:13:33 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 4a5d43964536093bf8932015bdd49b24c5b660ce

Copy link
Contributor

@sernan96 sernan96 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

상수화를 할 부분 외에는 딱히 문제가 없어 보입니다.
수고하셨습니다.

getChatSetting();
}, []);

const normalChatOptions = [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

상수화했으면 좋겠습니다!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

채팅 설정이 확장 가능성도 적고 해당 파일에서만 다룬다고 생각해서, 따로 상수 파일을 만들어서 관리하는게 오히려 더 복잡하지 않을까요?

{ label: '모두 차단', value: 'DENY_ALL' },
];

const whisperChatOptions = [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이것 또한 마찬가지입니다!

@whitecity01 whitecity01 merged commit 39141df into develop Nov 16, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants