fix: 처음 접속 시 서버에서 보내준 상태 반영, 여러 탭 사용 시 수동 자리비움 상태 변경 동작 모두 적용 #295
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
접속 중, 자리 비움, 부재 중 순으로 정렬
여러 탭을 띄웠을 때 설정된 상태가 초기에 반영되도록 수정 여러 탭을 띄웠을 때 수동 자리비움 상태가 유지되도록 수정
유저 상태가 오지 않는 경우 에러가 발생하기 때문에 기본값을 설정해 방지
중복으로 처리되어 memo를 수정할 때 모든 컴포넌트가 렌더링 되는 문제 수정
surinkwon
commented
Jun 13, 2024
Comment on lines
10
to
12
| const UserStateDisplay = ({ | ||
| status = "off", | ||
| }: { |
There was a problem hiding this comment.
서버에서 멤버의 status를 보내주지 않을 경우 발생하는 문제를 해결하기 위해 기본값을 설정했습니다.
There was a problem hiding this comment.
현재 백엔드 로직에 오류가 있는것 같은데 버그 재현이 안되네요. 다시 버그 발생시 오류를 인지할 수 있게하기 위해서 기본값을 설정하지 않는게 어떨까요?
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.
🎟️ 태스크
상태 변경 피드백 반영
✅ 작업 내용
🖊️ 구체적인 작업
자리 비움 수동 설정 동작