-
Notifications
You must be signed in to change notification settings - Fork 0
[deploy] v.1.2.9 운영서버로 배포 #311
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
Conversation
- notification DB 저장은 상위 트랜잭션 커밋시에 동기적으로 수행 - 상위 트랜잭션이 있을 경우에만 정상 동작하도록 Propagation.MANDATORY 제한 설정 - 푸시알림을 위한 이벤트 퍼블리시는 비동기적으로 수행 - 이벤트 리스너가 트랜잭션 커밋시에 이벤트를 받아서 fcm 서버로 푸시알림 보내는 구조는 유지
- notification DB 저장은 상위 트랜잭션 커밋시에 동기적으로 수행 - 상위 트랜잭션이 있을 경우에만 정상 동작하도록 Propagation.MANDATORY 제한 설정 - 푸시알림을 위한 이벤트 퍼블리시는 비동기적으로 수행 - 이벤트 리스너가 트랜잭션 커밋시에 이벤트를 받아서 fcm 서버로 푸시알림 보내는 구조는 유지
- 모든 이벤트에 title, content 추가 - 이벤트 리스너는 이벤트에 포함된 raw data 로 title, content 를 구성하는게 아니라, 이벤트에 포함된 title, content 정보를 그대로 사용하도록 하기 위해 이벤트 구조 수정
- 이벤트에 포함된 title, content 정보를 받아서 바로 사용하도록 수정 - title, content 의 생성 책임은 template enum 만 담당
- NotificationOrchestrator 를 호출하도록 수정 - NotificationOrchestrator 내부에서 알림센터 저장 + 이벤트 퍼블리시 수행
- NotificationOrchestrator 구현체를 의존하도록 수정
- 테스트 클래스 레벨에 트랜잭션이 걸려있으므로 teardown 메서드 삭제
- userJpaRepository 의 delete 전에 NotificationJpaRepository의 delete 를 선행하도록 코드 수정
- notification 패키지에서 알림센터의 title, content 생성을 담당하므로 notification/domain 하위로 이동
… 를 바라보도록 수정 (#296) - 일단 whitelist enum 객체를 도입 - 필요하다면 yml 에 명시하고, 환경변수로 주입받아도 될 듯
- NotificationQueryDto <-> response 매퍼 추가
- application/port/in/dto 의 MyRoomType enum 을 메서드 파라미터로 설정함으로써 service 내부에서 MyRoomType 검증과정 삭제
[feat] 알림센터 조회 api 개발
|
Important Review skippedMore than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review. 86 files out of 193 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits. You can disable this status message by setting the ✨ Finishing touches
🧪 Generate unit tests
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
#️⃣ 연관된 이슈
📝 작업 내용
v.1.2.9 업데이트 내용
📸 스크린샷
💬 리뷰 요구사항
📌 PR 진행 시 이러한 점들을 참고해 주세요