woowacourse-teams / 2024-mouda

17 stars 6 forks source link

채팅 알림 버그픽스 #687

Closed pricelees closed 2 days ago

pricelees commented 5 days ago

PR의 목적이 무엇인가요?

QA 과정에서 발생한 채팅 알림 관련 문제를 해결했습니다.

이슈 ID는 무엇인가요?

설명

1. 모임 정보를 수정 시 알림 문제

모임 정보 수정 시, 이름을 변경한 경우 변경된 이름으로 알림을 보내서 사용자에게 혼동을 줄 여지가 있다고 판단했습니다. (예시) 원래 모임 이름이 "모임" 이고 변경된 모임 이름이 "모임12" 라면, 사용자에겐 "모임12" 정보가 바뀜 ~ 으로 알림이 나갔는데, 사용자 입장에서는 "모임12" 라는 모임에 들어간 적은 없으니 헷갈릴 수 있음..

따라서 이 경우 기존 모임 이름을 받아 기존 모임의 이름을 넣도록 수정했습니다.

2. 채팅 알림 메시지 형식 수정

이전 수정에서, 채팅 알림의 경우 아래와 같은 형식으로 알림이 전송됐었습니다.

기존

일반 채팅 전송시

[모임 / 베팅 이름]
모우다에서 보냄
[보낸사람]: [내용]

장소 / 시간 확정시

[모임 / 베팅 이름]
모우다에서 보냄
장소가 [장소]로 확정되었어요!

푸시 알림만 보면 어떤 모임에서 누가 메시지를 보냈는지 확실하게 나타나긴 하지만, 알림 센터에서 조회 시 "장소가 [장소]로 확정되었어요!" 인 메시지만 나타나 사용자가 어떤 모임인지 식별하기 힘들겠다는 생각으로 아래와 같이 조금 더 세분화하는 방식으로 수정했습니다.

수정

일반 채팅 전송시

[모임 / 베팅 이름]
모우다에서 보냄
[보낸사람]: [내용]

장소 / 시간 확정시

[다락방 이름]
모우다에서 보냄
'[모임이름]' 시간이 '[시간 -> ~월 ~일 ~시 ~분]'으로 확정되었어요!

3. 채팅 알림 구현 방법 수정

새로 추가된 Author와 ChatType을 활용하는 방식으로 수정했습니다. 처음에 ChatType을 사용할 땐 NotificationType을 더 이상 사용하지 않을 것으로 기대했으나, 알림 쪽 처리에서의 문제로 완전히 격리시킬 수는 없었습니다.

4. 베팅에서의 채팅은 알림이 안오는 문제 해결

일반 채팅방에서의 알림은 잘 오는데, 베팅에서는 알림이 오지 않고 에러도 발생하지 않아 디버깅할 수도 없었는데요, 어찌저찌하여 일부 코드를 수정했고 이제 정상 작동이 될 것 같습니다 ~

질문 혹은 공유 사항 (Optional)