refactor/KD-59 : 일정 반환 enum을 한글로 보내주도록 수정했습니다.#309
Conversation
|
Important Review skippedAuto incremental 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 개요두 개의 일정 응답 클래스에서 submissionType 필드의 데이터 반환 방식을 변경했습니다. Swagger 예시 값은 영문 열거형명("MIDTHESIS")에서 한글 레이블("중간논문")로 업데이트되었으며, from() 팩토리 메서드는 enum의 name() 메서드 대신 getLabel() 메서드를 사용하도록 변경되었습니다. 변경 사항
예상 코드 리뷰 소요 시간🎯 2 (단순) | ⏱️ ~8분 Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
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. Comment |
Test Coverage Report
|
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## develop #309 +/- ##
==========================================
Coverage 86.51% 86.51%
Complexity 67 67
==========================================
Files 24 24
Lines 267 267
Branches 14 14
==========================================
Hits 231 231
Misses 24 24
Partials 12 12 Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
aics-api/src/main/java/kgu/developers/api/schedule/presentation/response/ScheduleSummaryResponse.javaaics-api/src/main/java/kgu/developers/api/schedule/presentation/response/ScheduleTypeContentResponse.java
🔇 Additional comments (2)
aics-api/src/main/java/kgu/developers/api/schedule/presentation/response/ScheduleSummaryResponse.java (1)
18-18: 변경 사항이 일관되게 적용되었습니다.
ScheduleTypeContentResponse와 동일하게submissionType필드가 enum 상수 대신 한글 레이블을 반환하도록 올바르게 수정되었습니다.Also applies to: 37-37
aics-api/src/main/java/kgu/developers/api/schedule/presentation/response/ScheduleTypeContentResponse.java (1)
11-11: 변경 사항이 PR 목표와 일치합니다.Swagger 예제와 실제 반환 값이 모두 한글 문자열("중간논문")을 사용하도록 올바르게 수정되었습니다.
SubmissionTypeenum의@Getter어노테이션으로 자동 생성된getLabel()메서드를 통해 enum 상수 대신 레이블을 반환하며, 스키마 예제와 실제 enum 값이 일치합니다.
...src/main/java/kgu/developers/api/schedule/presentation/response/ScheduleSummaryResponse.java
Outdated
Show resolved
Hide resolved
JangYeongHu
left a comment
There was a problem hiding this comment.
ScheduleListResponse의 submissionType도 수정이 필요해 보입니다
그리고 코드래빗 리뷰대로 일관성 있게 둘 다 한글 값을 반환하도록 만드는 것도 좋아 보입니다
그 외에는 잘 해주셨네요 수고하셨습니다~!
Summary
해당 PR에 대한 요약을 작성해주세요.
일정 타입을 반환할때 enum이 아닌 한글로 반환해주도록 설계하였습니다.
Tasks