Conversation
qkrwndnjs1075
commented
Jul 28, 2025
- gRPC 관련 비즈니스 로직 PR
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
| } catch (e: Exception) { | ||
| throw StatusException(Status.INTERNAL.withDescription("서버 오류")) | ||
| } | ||
| } |
There was a problem hiding this comment.
에러 처리는 비즈니스 로직 상에서 처리 되어지고 있기 때문에 제거 하였습니다.
| * @return gRPC 사용자 정보 응답 | ||
| */ | ||
| fun toGetUserInfoResponse(userResponse: InternalUserResponse): UserServiceProto.GetUserInfoResponse { | ||
| return UserServiceProto.GetUserInfoResponse.newBuilder() |
There was a problem hiding this comment.
.setReceiptCode() 이것도 추가해야 하는거 아닌가요 @qkrwndnjs1075
There was a problem hiding this comment.
근데 ?: 0L이 맞나 수험번호를 부여 받는 건데?
There was a problem hiding this comment.
아 생각해보니까 어짜피 receiptCode는 안 보내서 필요없음