채팅 알림 버그픽스 #687

pricelees · 2024-10-18T09:17:08Z

PR의 목적이 무엇인가요?

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

이슈 ID는 무엇인가요?

채팅 알림 버그픽스 #684

설명

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

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

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

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

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

기존

일반 채팅 전송시

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

장소 / 시간 확정시

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

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

수정

일반 채팅 전송시

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

장소 / 시간 확정시

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

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

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

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

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

질문 혹은 공유 사항 (Optional)

ay-eonii

고생하셨습니다 빠른 작업👍

코멘트 확인해주세용~~~

ay-eonii · 2024-10-18T11:51:53Z

backend/src/main/java/mouda/backend/moim/implement/sender/MoimNotificationSender.java

 		Darakbang darakbang = darakbangRepository.findById(moim.getDarakbangId())
-			.orElseThrow(IllegalArgumentException::new);
+			.orElseThrow(() -> new DarakbangException(HttpStatus.NOT_FOUND, DarakbangErrorMessage.DARAKBANG_NOT_FOUND));


package에 해당하는 예외를 던져주기로 했던 것 같아요

package에 해당하는 예외를 던져주기로 했던 것 같아요

해당 이유로 알림 타입에 대한 예외는 모임 에러를 사용했는데, 여기서는 좀 애매한 감이 있어서 다락방 예외를 사용했어요 ㅋㅋ 말씀주신대로 모임예외로 수정하고, 예외 메시지를 "모임에 해당되는 다락방을 찾을 수 없음" 으로 수정할게요 ~

ay-eonii · 2024-10-18T11:54:09Z

backend/src/main/java/mouda/backend/moim/implement/sender/MoimNotificationSender.java

-			MoimNotificationMessage.create(moim.getTitle(), notificationType),
-			getMoimUrl(darakbang.getId(), moim.getId()),
+			message,
+			getMoimUrl(darakbang.getId(), darakbang.getId()),


두개 다 다락방 아이디 맞나여??

두개 다 다락방 아이디 맞나여??

아뇨 ㅋㅋ 감사용 ㅋㅋ

pricelees

👍

pricelees · 2024-10-20T05:40:49Z

backend/src/main/java/mouda/backend/moim/implement/sender/MoimNotificationSender.java

 		Darakbang darakbang = darakbangRepository.findById(moim.getDarakbangId())
-			.orElseThrow(IllegalArgumentException::new);
+			.orElseThrow(() -> new DarakbangException(HttpStatus.NOT_FOUND, DarakbangErrorMessage.DARAKBANG_NOT_FOUND));


package에 해당하는 예외를 던져주기로 했던 것 같아요

해당 이유로 알림 타입에 대한 예외는 모임 에러를 사용했는데, 여기서는 좀 애매한 감이 있어서 다락방 예외를 사용했어요 ㅋㅋ 말씀주신대로 모임예외로 수정하고, 예외 메시지를 "모임에 해당되는 다락방을 찾을 수 없음" 으로 수정할게요 ~

pricelees · 2024-10-20T05:41:26Z

backend/src/main/java/mouda/backend/moim/implement/sender/MoimNotificationSender.java

-			MoimNotificationMessage.create(moim.getTitle(), notificationType),
-			getMoimUrl(darakbang.getId(), moim.getId()),
+			message,
+			getMoimUrl(darakbang.getId(), darakbang.getId()),


두개 다 다락방 아이디 맞나여??

아뇨 ㅋㅋ 감사용 ㅋㅋ

ay-eonii

굿굿~👍

ksk0605

고생했어요 상돌!
간단한 변경사항만 수정 부탁드려요 😀

ksk0605 · 2024-10-20T07:25:34Z

backend/src/main/java/mouda/backend/chat/domain/Author.java

+	public Author(long id, long memberId, String nickname, String profile) {
 		this.id = id;
+		this.memberId = memberId;


다락방멤버 id 인지 그냥 memberid인지 구분할 수 있으면 좋겠네요!

다락방멤버 id 인지 그냥 memberid인지 구분할 수 있으면 좋겠네요!

수정했어요 ~

ksk0605 · 2024-10-20T07:27:38Z

backend/src/main/java/mouda/backend/chat/util/ChatDateTimeFormatter.java

+public class ChatDateTimeFormatter {
+
+	// 2024-10-21 17:08 형식의 데이터를 10월 21일 17시 08분으로 변경
+	public static String formatDateTime(String dateTime) {
+		String[] splitDateTime = dateTime.split(" ");
+		LocalDate date = LocalDate.parse(splitDateTime[0]);
+		LocalTime time = LocalTime.parse(splitDateTime[1]);
+
+		return date.getMonthValue() + "월 "
+			+ date.getDayOfMonth() + "일 "
+			+ time.getHour() + "시 "
+			+ time.getMinute() + "분";
+	}
+}


생성할 필요가 없는 클래스의 생성자는 private으로 닫아주세요!

생성할 필요가 없는 클래스의 생성자는 private으로 닫아주세요!

넵 ~ 이왕 하는김에 조금 더 꼼꼼하게 처리해놨습니다 !

ksk0605 · 2024-10-20T07:28:38Z

backend/src/main/java/mouda/backend/moim/implement/finder/ChatRecipientFinder.java

 		return getNotificationRecipients(darakbangMemberStream, sender);
 	}

+	// todo: 구버전 채팅 기능 삭제시 같이 지워주세요.


다음 버전에서 지우도록하죠!

pricelees

하는김에 조금씩 더 다듬었어요 ~ㅎㅎ 확인 부탁요!

pricelees · 2024-10-20T09:00:09Z

backend/src/main/java/mouda/backend/chat/util/ChatDateTimeFormatter.java

+public class ChatDateTimeFormatter {
+
+	// 2024-10-21 17:08 형식의 데이터를 10월 21일 17시 08분으로 변경
+	public static String formatDateTime(String dateTime) {
+		String[] splitDateTime = dateTime.split(" ");
+		LocalDate date = LocalDate.parse(splitDateTime[0]);
+		LocalTime time = LocalTime.parse(splitDateTime[1]);
+
+		return date.getMonthValue() + "월 "
+			+ date.getDayOfMonth() + "일 "
+			+ time.getHour() + "시 "
+			+ time.getMinute() + "분";
+	}
+}


생성할 필요가 없는 클래스의 생성자는 private으로 닫아주세요!

넵 ~ 이왕 하는김에 조금 더 꼼꼼하게 처리해놨습니다 !

pricelees · 2024-10-20T09:02:02Z

backend/src/main/java/mouda/backend/chat/domain/Author.java

+	public Author(long id, long memberId, String nickname, String profile) {
 		this.id = id;
+		this.memberId = memberId;


다락방멤버 id 인지 그냥 memberid인지 구분할 수 있으면 좋겠네요!

수정했어요 ~

ksk0605

수정 사항 확인했습니다! 고생했어요~

pricelees added 3 commits October 18, 2024 16:44

refactor: 모임 정보 수정시 알림 메시지에서 수정 전 모임 이름을 사용하도록 수정

43fd7bb

refactor: 채팅 알림 구현 방법 및 메시지 형식 수정

8ff90ec

refactor: 채팅 알림 메시지 세분화

17ee186

pricelees requested review from ay-eonii, ksk0605, hoyeonyy and Mingyum-Kim and removed request for ay-eonii October 18, 2024 09:20

ay-eonii requested changes Oct 18, 2024

View reviewed changes

pricelees added 2 commits October 20, 2024 14:41

refactor: 모임에서의 다락방 조회 예외 메시지 추가

625004e

fix: 모임 ID 대신 다락방 ID를 사용하는 오타 수정

44a4c0a

pricelees commented Oct 20, 2024

View reviewed changes

ay-eonii self-requested a review October 20, 2024 06:02

ay-eonii approved these changes Oct 20, 2024

View reviewed changes

ksk0605 requested changes Oct 20, 2024

View reviewed changes

pricelees added 6 commits October 20, 2024 17:03

refactor: 참여자가 입력된 DarakbangMember인지 확인하는 메서드 추가

68f15fa

refactor: memberId 필드 추가에 따른 Author 필드명 세분화(id -> darakbangMember)

7e93055

refactor: Recipient 필드 final 지정 및 빌더 추가

f883d4f

refactor: CommentRecipient를 Map을 가진 일급 컬렉션으로 수정

b491a91

refactor: ChatDateTimeFormatter 세분화

ac98c87

fix: DateTimeFormatter 사용 제거

aa86f72

pricelees commented Oct 20, 2024

View reviewed changes

Merge branch 'develop-backend' into fix/#684

325112c

ksk0605 approved these changes Oct 20, 2024

View reviewed changes

pricelees merged commit 9f15539 into develop-backend Oct 21, 2024
1 check passed

ay-eonii deleted the fix/#684 branch October 24, 2024 07:49

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

채팅 알림 버그픽스 #687

채팅 알림 버그픽스 #687

pricelees commented Oct 18, 2024 •

edited

Loading

ay-eonii left a comment

ay-eonii Oct 18, 2024

pricelees Oct 20, 2024

ay-eonii Oct 18, 2024

pricelees Oct 20, 2024

pricelees left a comment

pricelees Oct 20, 2024

pricelees Oct 20, 2024

ay-eonii left a comment

ksk0605 left a comment

ksk0605 Oct 20, 2024

pricelees Oct 20, 2024

ksk0605 Oct 20, 2024

pricelees Oct 20, 2024

ksk0605 Oct 20, 2024

pricelees left a comment

pricelees Oct 20, 2024

pricelees Oct 20, 2024

ksk0605 left a comment

채팅 알림 버그픽스 #687

채팅 알림 버그픽스 #687

Conversation

pricelees commented Oct 18, 2024 • edited Loading

PR의 목적이 무엇인가요?

이슈 ID는 무엇인가요?

설명

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

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

기존

수정

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

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

질문 혹은 공유 사항 (Optional)

ay-eonii left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

pricelees left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ay-eonii left a comment

Choose a reason for hiding this comment

ksk0605 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

pricelees left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ksk0605 left a comment

Choose a reason for hiding this comment

pricelees commented Oct 18, 2024 •

edited

Loading