Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/#895 채팅창에서 메세지 알림 클릭 시 버그 #896

Conversation

ki960213
Copy link
Collaborator

@ki960213 ki960213 commented Jan 7, 2024

#️⃣ 연관된 이슈

close : #895

📝 작업 내용

채팅창에서 다른 채팅창의 메세지 알림이 왔을 때 그 알림 클릭하면 다른 채팅창으로 이동하도록 수정했습니다. 원래는 이동하지 않았습니다. 제가 리팩터링 하면서 잘못 구현했습니다.

그리고 프로필 화면 간 다음 거기서 쪽지 보내면 방금 보낸 쪽지가 보이지 않았습니다. 이유는 onNewIntent()가 호출되었기 때문입니다. 따라서 로그인한 사용자가 메세지 보내서 onNewIntent()가 호출된 거라면 새로고침하도록 수정하였습니다. 근데 이건 디자인을 수정해야 할 문제 같습니다. 프로필 화면에서 메세지 다이얼로그 띄워서 메세지를 보내는 것보단 그냥 채팅창 화면으로 이동할 수 있게 하는 게 더 나을 것 같아요. 만약 한 번도 채팅하지 않아서 roomId가 없다면 카카오톡처럼 빈 화면을 보여주게 하면 좋을 것 같아요. 이 부분은 어떻게 생각하시나요?

스크린샷 (선택)

예상 소요 시간 및 실제 소요 시간 (일 / 시간 / 분)

예상 소요 시간 : 5분
실제 소요 시간 : 40분

💬 리뷰어 요구사항 (선택)

@ki960213 ki960213 added 버그 개발자가 의도하지 않은 상황 Android 안드로이드 관련 이슈 High Priority 리뷰 우선순위가 높은 PR labels Jan 7, 2024
@ki960213 ki960213 self-assigned this Jan 7, 2024
@ki960213 ki960213 force-pushed the fix/#895-채팅창에서_메세지_알림_클릭_시_버그 branch from 1e3283d to a7c65f1 Compare January 7, 2024 17:34
- isSendByLoginUser -> isSendNewMessageByLoginUser
@ki960213 ki960213 closed this Feb 7, 2024
@ki960213 ki960213 deleted the fix/#895-채팅창에서_메세지_알림_클릭_시_버그 branch February 7, 2024 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Android 안드로이드 관련 이슈 High Priority 리뷰 우선순위가 높은 PR 버그 개발자가 의도하지 않은 상황
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant