[Feat/#48] 채팅 페이지 QA #49
Merged
+138
−20
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
ljh0608
approved these changes
Feb 7, 2025
Comment on lines
+3
to
+8
| type AcceptRejectButtonDisplayContextType = { | ||
| isVisible: boolean; | ||
| show: () => void; | ||
| hide: () => void; | ||
| }; | ||
|
|
There was a problem hiding this comment.
p2) provider가 많아지면서 hide를 의미하는 다양한 provider가 생긴것 같아요 이렇게 되면 컴포넌트 내에서 불러올 때 hide의 이름을 재정의 해주어야하고, 그러면 여러 컴포넌트에서 각각이 다른 이름을 사용하게 되어서 혼란을 일으킬 수 있을 것 같아요!
이부분을 Context 의미에 맞는 이름으로 변경하면 좋을 것 같아요!
hide를 각각 다르게 정의하고 사용하는 부분 코드 예시
hide: hideTextField,
focus: focusTextField,
} = useTextFieldInChatDisplayContext();
const { hide: hideAcceptRejectButtons }
There was a problem hiding this comment.
프로바이더들을 동일한 인터페이스로 맞춰 사용하기 편하게 만드려고 했는데 그런 단점도 있겠네 재훈이는 useQuery 커스텀 훅을 만드는 경우에도 아래 처럼 이름을 변경해서 반환하나?
const { customData } = useCustomQuery
Comment on lines
+15
to
+17
| <AcceptRejectButtonDisplayProvider> | ||
| <ChatRoom /> | ||
| </AcceptRejectButtonDisplayProvider> |
There was a problem hiding this comment.
p3) 준근님만의 Provider를 도입하는 기준이 궁금해요!
There was a problem hiding this comment.
기준이 있지는 않고 복잡한 상태를 좁은 화면에서 관리해야 하는 문제가 있어서 reducer와 context를 같이 사용했어
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
이슈 번호
closes #48
작업한 내용
디스코드 qa냥 채널에 보고된 채팅 페이지 관련 QA
작업 결과
.
비고
시간이 없어서 티켓을 못 나누고 한 번에 작업했어 ㅜ 커밋 메시지로 변경 사항 구분해 놨으니 참고