Skip to content

Conversation

@modzivv
Copy link
Member

@modzivv modzivv commented Jul 31, 2025

📝 Description

이전 PR(#110)에서 누락된 Notification 컨트롤러의 userId 참조 수정 보완 및 불필요한 BigInt 로직 개선

Fixes #111

Related #110

⚙️ Type

  • 기능 추가 (새로운 API, 서비스 로직 등)
  • 버그 수정 (예외 처리, 동작 오류 등)
  • 리팩토링 (코드 정리, 로직 개선 등)
  • 문서 수정 (README, Swagger, 주석 등)
  • 테스트 코드 추가/수정
  • 의존성 추가/수정

📂 Summary of Changes

  • src/notification/controller/notification.controller.js, src/notification/fcm/controller/push.controller.js: req.user.id -> req.user.userId로 수정
  • src/notification/repository/notification.repository.js: 불필요한 BigInt() 호출 제거 (이미 BigInt로 전달된 값 재변환 방지)

👀 To Reviewer

  • 이전 PR에서 Notification 컨트롤러 userId 참조 수정을 누락한 것을 뒤늦게 발견해 곧바로 반영했습니다
  • 같은 이슈로 다시 PR을 올리게 되었는데 번거롭게 해드려 죄송합니다... 😓

✅ PR Checklist

  • 커밋 메시지 컨벤션을 준수했습니다.
  • 코드 컨벤션을 준수했습니다.
  • 기능이 정상 동작하는지 테스트했습니다.
  • Swagger 문서를 최신 상태로 반영했습니다. (필요 시)

@modzivv modzivv self-assigned this Jul 31, 2025
@modzivv modzivv added the bug Something isn't working label Jul 31, 2025
Copy link
Collaborator

@bkw535 bkw535 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다!

@taeyoung0524 taeyoung0524 self-requested a review August 1, 2025 00:07
Copy link
Collaborator

@taeyoung0524 taeyoung0524 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다!

@modzivv modzivv merged commit c2ccfec into develop Aug 1, 2025
1 check passed
@modzivv modzivv deleted the fix/111-userid-field branch August 1, 2025 04:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Notification 컨트롤러 userId 참조 누락 수정

4 participants