[FIX] BigInt(undefined) 서버 오류 해결 #110
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
리뷰 및 알림 기능 API에서
BigInt(undefined)변환으로 인한 서버 500 에러 해결Fixes #104
Related #12 #30 #39
⚙️ Type
📂 Summary of Changes
src/common/errors/notification.errors.js: 에러 코드(N012-N013) 추가src/common/errors/review.errors.js: 에러 코드(R011-R014) 추가src/notification/controller/notification.controller.js,src/notification/fcm/controller/push.controller.js,src/review/controller/review.controller.js: 필수 파라미터 누락 시 커스텀 에러 반환 로직 추가src/review/controller/review.controller.js:req.user.id->req.user.userId로 수정 및 사용자 ID 접근 방식 통일👀 To Reviewer
req.user.id) undefined 값이 들어가면서BigInt(undefined)에러가 발생하였으나, 이제는req.user.userId를 사용하여 정상적으로 id값이 전달되도록 개선하였습니다.notificationId등)가 누락될 경우 기존에는 서버에서 500 에러가 발생했으나,이제는 명확한 커스텀 에러(
NotificationIdRequiredError등)를 반환하도록 개선했습니다.✅ PR Checklist