-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
✨ [Feature] Admin 공유일정 수정 API 및 테스트 코드 작성 #1059 #1086
The head ref may contain hidden characters: "1059-feature-admin-\uACF5\uC720\uC77C\uC815-\uC218\uC815"
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #1086 +/- ##
============================================
+ Coverage 74.42% 74.51% +0.09%
- Complexity 1227 1230 +3
============================================
Files 399 401 +2
Lines 5524 5545 +21
Branches 440 440
============================================
+ Hits 4111 4132 +21
- Misses 1220 1222 +2
+ Partials 193 191 -2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@@ -240,7 +240,7 @@ public enum ErrorCode { | |||
CALENDAR_BEFORE_DATE(400, "CA201", "종료 시간이 시작 시간보다 빠를 수 없습니다."), | |||
CALENDAR_AFTER_DATE(400, "CA202", "시작 시간이 종료 시간보다 늦을 수 없습니다."), | |||
CALENDAR_EQUAL_DATE(400, "CA203", "시작 시간과 종료 시간이 같을 수 없습니다."), | |||
CALENDAR_AUTHOR_NOT_MATCH(400, "CA205", "전제하지 않는 사용자입니다."), | |||
CALENDAR_AUTHOR_NOT_MATCH(400, "CA205", "존제하지 않는 사용자입니다."), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
존재하지 않는 사용자로 바꿔주시기 바랍니다. 😗 403으로 안해주셨던 이유가 있을까요?! git pull 받으셔서 안하신건가요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
깔끔하고 군더더기없는 코드 잘보고갑니다. 🍀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🍀 세이킴포저 🍀 미션컴플리트
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
public PageResponseDto<PublicScheduleAdminResDto> findAllByClassification( | ||
DetailClassification detailClassification, int page, int size) { | ||
public PageResponseDto<PublicScheduleAdminResDto> findAllByClassification(DetailClassification detailClassification, | ||
int page, int size) { | ||
|
||
Pageable pageable = PageRequest.of(page - 1, size, | ||
Sort.by(Sort.Order.asc("status"), Sort.Order.asc("startTime"))); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
벌써 정렬까지 대단하십니다.
📌 개요
💻 작업사항
💡Issue 번호