Skip to content

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 30 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 수정
* [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 수정

---------

Co-authored-by: soohyun <[email protected]>
Co-authored-by: soohyun <[email protected]>
[Fix/#198] 페이로드 부분 수정
[Merge] 댓글알림 버그 해결
sss4920 and others added 30 commits February 5, 2024 20:09
[Feat] 개선된 CI/CD 기반으로 마이페이지 API 구조변경 사항 재차 PR
[Refactor]#232 절약 누적횟수 추가
[Fix] 피드생성시 레벨업 달성여부 bool 값 responseBody에 담게끔 수정
[Refactor/#236] 첫로그인 메시지 변경
[Feat]#238 feature: 전체 푸시알림메시지 발송
* [Feat] 마이페이지 응답값 변경 (+가입일, -100일 절약 금액)

* [Feat] 마이페이지 응답값 가입일 -> 가입 디데이로 변경
* [Feat] 소셜 로그인 관련 로그 삭제

* [Feat] goal 관련 로직 삭제
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.

4 participants