[FIX] Notification 컨트롤러 userId 참조 누락 수정 #112
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 Description
이전 PR(#110)에서 누락된 Notification 컨트롤러의
userId참조 수정 보완 및 불필요한 BigInt 로직 개선Fixes #111
Related #110
⚙️ Type
📂 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
userId참조 수정을 누락한 것을 뒤늦게 발견해 곧바로 반영했습니다✅ PR Checklist