issues
search
connect-foundation
/
2019-08
✉실시간 그룹 채팅 서비스 ✉
MIT License
12
stars
4
forks
source link
현재 Thread 와 상관없는 reply 와 로그인 화면에 알림 컴포넌트 노출되는 이슈
#225
Closed
raccoonback
closed
4 years ago
raccoonback
commented
4 years ago
Description and Motivation
다른 포스트에 대한 댓글도 Thread 에 노출되는 이슈
현재 Thread 에 노출되는 post.id 와 새로운 댓글의 post.id 비교하여 렌더링 여부를 결정하도록 변경
채널 첫 입장시 댓글에 대한 소켓 이벤트를 처리하지 못하는 이슈
채널 입장시 소켓 replyPost 이벤트 구독하는 로직 추가하여 해결
회원가입 및 로그인 페이지에도 invite-alarm 컴포넌트가 노출되는 이슈
로그인된 경우에만 invite-alarm 컴포넌트 노출하도록 변경하여 해결
로그인 후 Snug 입장하지 않으면 로그아웃되지 않는 이슈
Snug 입장하지 않아 Profile 쿠키가 존재하지 않는다
따라서 Snug 입장하지 않고 로그아웃할 경우, API 서버의 "Profile 쿠키 존재 여부 검증"하는 미들웨어에서 에러가 발생한다
"Profile 쿠키 존재 여부 검증" 제거하고, 올바르게 Profile 쿠키를 제거하도록 변경하여 해결
Types of changes
[ ] Docs change
[ ] dependency upgrade
[ ] refactoring
[X] Bug fix
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Review
Checklist
[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
Description and Motivation
다른 포스트에 대한 댓글도 Thread 에 노출되는 이슈
채널 첫 입장시 댓글에 대한 소켓 이벤트를 처리하지 못하는 이슈
회원가입 및 로그인 페이지에도 invite-alarm 컴포넌트가 노출되는 이슈
로그인 후 Snug 입장하지 않으면 로그아웃되지 않는 이슈
Types of changes
Checklist