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

[Feature] 알림 전송 구현 및 알림 목록 조회를 위한 Entity 생성 #192

Merged
merged 4 commits into from
May 13, 2024

Conversation

kyeong-hyeok
Copy link
Member

💡 연관된 이슈

close #189

📝 작업 내용

  • 이동봉사자, 모집자의 알림을 저장하는 Entity 생성
  • 봉사자 알림 전송 및 저장 구현 (이동봉사 승인, 반려, 완료)
  • 모집자 알림 전송 및 저장 구현 (이동봉사 신청, 신청 취소, 후기 등록)

💬 리뷰 요구 사항

Notification Entity

  • 봉사자와 모집자의 Notification Entity를 구분했고, 알림을 봉사자는 VolunteerNotification에 모집자는 IntermediaryNotification에 저장하여 알림 목록 조회 시 봉사자id, 모집자id로 불러올 수 있도록 구현했습니다! (하나의 Entity에 구현할 수도 있었지만 분리하는 것이 관리와 조회면에서 이점이 있다고 생각했습니다.)
  • Notification 필드에 isRead를 두어 해당 알림의 읽음 여부를 판단할 수 있도록 했습니다.

NotificationMessage (알림 제목, 내용)

  • NotificationMessage를 통해서 알림 메시지에 대해 유지보수가 용이하게 개발했습니다! 이동봉사자의 닉네임에 따라 변경되는 메시지의 내용을 메서드로 통일성 있게 구현했습니다

알림 전송 및 저장

  • 알림을 전송하는 메서드 내에 알림을 저장하는 로직을 추가하여 알림을 전송하고 해당 알림을 목록에 저장할 수 있도록 구현하였습니다. 이는 봉사자와 모집자의 Notification Entity를 분리했기 때문에 메서드를 분리하여 사용했습니다.

이전에 Fcm Token을 이용해 알림이 전송되는지 테스트는 진행했지만, 이번 작업에서 트리거 발생 시 Fcm Token을 통한 테스트를 따로 진행하지 않아 프론트와 확인해 보면 좋을 것 같습니다.

@kyeong-hyeok kyeong-hyeok added ✨ Feature 기능 개발 Priority: Medium 우선순위 중간 🐯 Koeyhk 담당자 labels May 11, 2024
@kyeong-hyeok kyeong-hyeok requested a review from hojeong2747 May 11, 2024 08:57
Copy link
Member

@hojeong2747 hojeong2747 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

도메인 분리해서 이해하기 쉽고 유지보수가 용이하게 구현하신 것 같아요! 실제 테스트 성공 기원 :)

@@ -6,7 +6,18 @@
@Getter
@RequiredArgsConstructor
public enum NotificationMessage {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NotificationMessage라는 enum 클래스로 관리하는 방식이 아주 좋은 것 같습니다. 동적인 값만 따로 붙여서 사용할 수 있도록 구현하셔서 유지보수하기 좋을 것 같아요!

@kyeong-hyeok kyeong-hyeok merged commit 57a8d4d into develop May 13, 2024
1 check passed
@kyeong-hyeok kyeong-hyeok deleted the feat/189-notification-send-and-save branch May 13, 2024 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 🐯 Koeyhk 담당자 Priority: Medium 우선순위 중간
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] 알림 전송 구현 및 알림 목록 조회를 위한 Entity 생성
2 participants