Skip to content

Pr Feat/29 fcm aop notification #129

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.

Sign up for GitHub

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

Jump to bottom
Merged
toychip merged 63 commits into from
Apr 25, 2024
Merged

Pr Feat/29 fcm aop notification #129

toychip merged 63 commits into from
Apr 25, 2024

Conversation

@toychip
Copy link
Member

@toychip toychip commented Apr 25, 2024 •
edited
Loading

Issue number and Link

Summary

FCM과 AOP를 사용하여 요구사항에 맞게 알림기능을 구현하였습니다.

PR Type

  • Feature
  • Bugfix
  • Refactoring
  • Documentation
  • Other

Other Information

  1. 프로필 좋아요 기능 추가 후 알림 기능 추가
  2. 알림 전송 실패시 추가 로직
  3. 실제 알림 전송시 TargetClassName, TargetClassIndex 알림 전송 문구 검토
  4. CartRepository addParticipant() 쿼리로 추가하는 것이 아닌 엔티티로 연관관계 매핑 후 생성하도록 수정

toychip added 30 commits April 6, 2024 02:22
@toychip
feat: 알림 애노테이션 정의 (#126)
96f999b
@toychip
feat: Aspect (#126)
5c302ea
@toychip
fix: 외부에서 Message를 파악하기 위해 접근제어자 변경 (#126)
ba18240
@toychip
feat: Firebase FCM 의존성, key.json, gitignore 추가 (#126)
20c10e2
@toychip
feat: FCM Configuration (#126)
a6224c6
@toychip
refactor: aop 코드 경로 변경 (#126)
8084336 
notification에서 notification.aop로 변경
@toychip
feat: fcm token field 추가 (#126)
5b37794
@toychip
feat: fcm token 등록 Api 추가 (#126)
28566de
@toychip
feat: Notification Entity 추가 (#126)
29730fb
@toychip
feat: FCMManager를 사용하여 푸쉬 알림 전송 (#126)
58c744f
@toychip
feat: FCMNotificationService (#126)
6c07788 
fcm token을 활용하여 검증 후 FCMManager에게 알림을 전달하는 서비스
@toychip
fix: NotificationAspect Advisor logic (#126)
7f66bf6 
알림을 보낼 대상, 클래스명, 메서드명을 전달하는 어드바이저로 수정
@toychip
refactor: FCMNotificationService 분리 (#126)
b9c347e 
FCMNotificationService에서 알림 카테고리 판단과 저장의 역할을 NotificationService로 이동
@toychip
feat: Notification 저장 로직 추가 (#126)
d88c4a5
@toychip
refactor: FCMNotificationService, NotificationService, FCMManager ren…
ec765d5 
…ame (#126)

FCMNotificationService -> NotificationDispatcher (알림 발송 역할), FCMManager -> FCMService (FCM Api 호출 역할), NotificationService -> NotificationRegistry (알림 저장, 판단 역할)
@toychip
feat: 꿀팁공유해요 좋아요, 스크랩 알림 처리 (#126)
2a1fa28 
LikeService, ScrapService annotation, NotificationRegistry.determineNoticategory() 판단 추가
@toychip
feat: 댓글 알림 Advisor 추가 (#126)
821da1e 
댓글은 반환 값을 사용하여 해결할 수 없고 클래스명과 메서드명으로 확인할 수 없으므로 별도의 댓글 알림 Adivsor 추가
@toychip
feat: 댓글 알림 기능 추가 (#126)
f799c7b 
모든 댓글 중 알림 기능에 해당하는 댓글에만 알림 발송
@toychip
feat: 알림 대상이 1명이 아니므로 작성자 판단 값 추가 (#126)
e631792
@toychip
fix: 잘못된 댓글 Aspect 수정 (#126)
409fbfc
@toychip
feat: 댓글 생성시 게시글 작성자와 현재 사용자가 일치하는지 여부 값 추가 (#126)
4710070
@toychip
feat: 답글 생성시 댓글글 작성자와 현재 사용자가 일치하는지 여부 값 추가 (#126)
b68ccd2
@toychip
feat: 댓글 타겟 필터링 fetchjoin repository (#126)
ee61194
@toychip
refactor: 댓글 생성시 알림 타겟 필터 분리 (#126)
c19034e
@toychip
fix: 누락 트랜잭션 추가, 미사용 import 제거, 의미없는 개행 제거 (#126)
1463d03
@toychip
fix: 댓글 조회 로직 수정 (#126)
e74c5de 
댓글을 삭제하면 상태 값에 삭제로 두고 내용을 바꾸기 때문에 삭제된 것을 필터링하는 것 제거
@toychip
fix: 부모 댓글이 삭제된 경우 대댓글 생성 불가하도록 검증 (#126)
aab38fe
@toychip
fix: 질문해요 댓글 조회시 삭제된 것 필터링 로직 제거 (#126)
d2d995c 
댓글은 삭제된 댓글입니다로 나와야하므로 삭제된 것을 안보여주면 안됨
@toychip
BREAKING CHANGE: 잘못된 fetchJoin, distinct() 모두 수정 (#126)
ae846e4 
컬렉션이 2개 즉, xxxToMany가 2개일 때 하나만 fetch join하여 의미가 없던 것을 배치 사이즈로 해결
@toychip
feat: 댓글 생성시 게시글의 작성자 쿼리, 답글 생성시 댓글 작성자 쿼리 fetch join (#126)
6d5d8ea
toychip added 23 commits April 20, 2024 22:22
@toychip
feat: AOP가 맡은 역할에서 EventListener를 사용하여 Comment Entity 트랜잭션 분리 (#126)
fcc22b5
@toychip
refactor: 사용되지 않는 로그 제거 (#126)
159c485
@toychip
fix: 댓글 단건 조회 잘못된 예외 처리 (#126)
ac0c31d
@toychip
feat: NotificationListener 도입 (#126)
6539f47
@toychip
fix: NotificationRegistry -> NotificationService 이동 순환참조 해결 (#126)
7c99df5
@toychip
feat: 알림 전송 성공 여부값 추가 (#126)
362d03e
@toychip
feat: 비동기로 처리하여 트랜잭션 별도 처리 (#126)
02439f3 
알림 전송과 같은 트랜잭션을 분리하여 영향을 주지 않도록 변경
@toychip
refactor: 전송 성공 반환 이메일로 변경 (#126)
71e7e48
@toychip
refactor: 댓글일 때, 게시글일 때의 TargetFinder 분리 (#126)
45993e7
@toychip
refactor: 미사용 쿼리 제거 (#126)
780d80c
@toychip
feat: 꿀팁공유해요 좋아요, 스크랩시 작성자에게 알림 (#126)
4f3c098
@toychip
feat: 우리동네(소통해요 한정) 스크랩시 작성자에게 알림 (#126)
78c5070
@toychip
fix: Cart 조회시 Writer Null 해결 (#126)
273d6d4
@toychip
refactor: 중복된 쿼리 사용 메서드 통일 (#126)
b8f55d9
@toychip
feat: 공구 참여시 작성자에게 알림 (#126)
fa5a45d
@toychip
feat: 어떠한 이벤트로 인해 알림이 전송됐을 때 본인이면 알림 전송 x (#126)
669451b 
본인 게시글에 본인이 댓글 달면 전송 x
@toychip
fix: 예외가 발생해도 알림 엔티티를 생성하기 위해 Post Like, Scrap 등 리스너 사용 (#126)
f02aa43
@toychip
feat: 우리동네 알림 전송 애노테이션 적용 (#126)
5cc18f2
@toychip
refactor: 경로 분기, typo, 미사용 Import 제거 (#126)
91e76af
@toychip
refactor: event record로 변경, static factory method 사용 (#126)
91a8d36
@toychip
feat: 질문해요 게시글에서 댓글 좋아요시 댓글 작성자에게 알림 (#126)
23df964
@toychip
feat: 알림 엔티티 targetClassType, targetClassIndex 필드 추가 (#126)
6b1d25f
@toychip
feat: 본인 알림 조회 API, 기능 추가 (#126)
4d87636
@toychip toychip added ✨ Feature 새로운 기능 추가 임준형 labels Apr 25, 2024
@toychip toychip linked an issue Apr 25, 2024 that may be closed by this pull request
feat: FCM + AOP를 활용한 알림기능 #126
Closed
13 tasks
@toychip toychip merged commit 475154f into main Apr 25, 2024
@toychip toychip mentioned this pull request Jun 11, 2024
Closed
13 tasks
@toychip toychip deleted the Feat/29-Fcm-Aop-Notification branch October 17, 2024 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

✨ Feature 새로운 기능 추가 임준형

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

feat: FCM + AOP를 활용한 알림기능

2 participants

@toychip