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

revert fcm 배포 #191

Merged
merged 6 commits into from
Nov 17, 2023
Merged

revert fcm 배포 #191

merged 6 commits into from
Nov 17, 2023

Conversation

funnysunny08
Copy link
Contributor

🚩 관련 이슈

📋 구현 기능 명세

  • [x]

📌 PR Point

  • 무슨 이유로 어떻게 코드를 변경했는지

  • 어떤 부분에 리뷰어가 집중해야 하는지

  • 개발하면서 어떤 점이 궁금했는지

📸 결과물 스크린샷

@Pattern(regexp = "^[\\S][가-힣a-zA-Z0-9\\s]{0,20}$", message = "위니 피드 제목 형식에 맞지 않습니다.")

🛠️ 테스트

  • 테스트

🚀 API Endpoint

  • /

sss4920 and others added 6 commits October 29, 2023 17:49
[Refactor] feed 쪽 알림 로직 중복코드 제거
* [Feat] fcm service 구현하기

* #159 [Feat] createMessage queue

* [Feat] 좋아요, 댓글 생성시 알림 발송

* [Feat] 메시지 큐 실험 성공, fcm만 되는지 확인필요

* [Feat] 푸시sender transactional 제거

* [Feat] fcm토큰 준비

* [Feat] rabbitMQ server ec2연결 및 token response 요구사항 반영수정

* [Feat] updateFcmtoken 구현 및 notification success notFound -> no content수정

* [Merge] merge confict 해결

* [Fix] 닉네임 수정 로직이랑 줄을 헷갈려서 버그 수정

* [FIX] merge conflict

* [FIX] stash 결과 적용

* [Feat] 동의여부 기능 추가 및 fcm 메시지 전송 변경

* [Feat] 댓글, 좋아요 알림로직에 동의 여부 수정 및 예외처리

* [Fix] notificationresponse Id 수정

* [Fix] logic 수정
@funnysunny08 funnysunny08 merged commit 81c18eb into main Nov 17, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants