Skip to content
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

Merged
merged 2 commits into from
Dec 30, 2024

Conversation

seyeon22222
Copy link
Contributor

📌 개요

  • Admin 공유일정 수정 API 작성했습니다.

💻 작업사항

  • Admin 공유일정 수정 API 작성
  • Admin 공유일정 수정 API 테스트 코드 작성

💡Issue 번호

@seyeon22222 seyeon22222 linked an issue Dec 30, 2024 that may be closed by this pull request
2 tasks
Copy link

codecov bot commented Dec 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.51%. Comparing base (a71cfc0) to head (b986936).
Report is 11 commits behind head on dev.

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     
Flag Coverage Δ
integrationTest 69.52% <ø> (+0.13%) ⬆️
unitTest 35.85% <ø> (-0.12%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@@ -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", "존제하지 않는 사용자입니다."),
Copy link
Contributor

@wonies wonies Dec 30, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

존재하지 않는 사용자로 바꿔주시기 바랍니다. 😗 403으로 안해주셨던 이유가 있을까요?! git pull 받으셔서 안하신건가요?

wonies
wonies previously approved these changes Dec 30, 2024
Copy link
Contributor

@wonies wonies left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

깔끔하고 군더더기없는 코드 잘보고갑니다. 🍀

Copy link
Contributor

@wonies wonies left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍀 세이킴포저 🍀 미션컴플리트

Copy link
Contributor

@taehyeon3 taehyeon3 left a 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")));
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

벌써 정렬까지 대단하십니다.

@seyeon22222 seyeon22222 merged commit 48ec5bb into dev Dec 30, 2024
5 checks passed
@seyeon22222 seyeon22222 deleted the 1059-feature-admin-공유일정-수정 branch December 30, 2024 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ [Feature] Admin 공유일정 수정
3 participants