Skip to content

Conversation

@hd0rable
Copy link
Member

#️⃣ 연관된 이슈

closes #이슈번호

📝 작업 내용

v.1.2.9 업데이트 내용

  1. [Refactor] ver 1.3.0 모임방 사용성 개선 작업
  2. [feat] 알림센터 조회 api 개발
  3. [refactor] 소셜 로그인 동적 리다이렉트되도록 변경

📸 스크린샷

💬 리뷰 요구사항

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

📌 PR 진행 시 이러한 점들을 참고해 주세요

* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

seongjunnoh and others added 30 commits September 12, 2025 04:08
- 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 내부에서 알림센터 저장 + 이벤트 퍼블리시 수행
- 테스트 클래스 레벨에 트랜잭션이 걸려있으므로 teardown 메서드 삭제
- userJpaRepository 의 delete 전에 NotificationJpaRepository의 delete 를 선행하도록 코드 수정
- notification 패키지에서 알림센터의 title, content 생성을 담당하므로 notification/domain 하위로 이동
… 를 바라보도록 수정 (#296)

- 일단 whitelist enum 객체를 도입
- 필요하다면 yml 에 명시하고, 환경변수로 주입받아도 될 듯
seongjunnoh and others added 25 commits September 17, 2025 04:11
- NotificationQueryDto <-> response 매퍼 추가
- application/port/in/dto 의 MyRoomType enum 을 메서드 파라미터로 설정함으로써 service 내부에서 MyRoomType 검증과정 삭제
@coderabbitai
Copy link

coderabbitai bot commented Sep 22, 2025

Important

Review skipped

More 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 reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

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.

  • Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
  • Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

Test Results

474 tests  ±0   474 ✅ ±0   43s ⏱️ -1s
140 suites ±0     0 💤 ±0 
140 files   ±0     0 ❌ ±0 

Results for commit d96c12e. ± Comparison against base commit 9cd560c.

@hd0rable hd0rable merged commit 7f6b333 into main Sep 22, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants