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

🔨 [Refactoring] Admin 공유일정 조회 API -> 분류별 일정조회 API로 변경 #1091 #1093

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

seyeon22222
Copy link
Contributor

📌 개요

  • Admin 공유일정 조회 API -> 분류별 일정조회 API로 변경했습니다.

💻 작업사항

  • Admin 공유일정 조회 API -> 분류별 일정조회 API로 변경

✅ 변경로직

  • totalschedule 패키지 및 세부 패키지 생성했습니다.
  • publicScheduleAdmin에 있던 함수들을 totalScheduleController로 이동했습니다.
  • admin관련 securityConfig파일 수정했습니다.

💡Issue 번호

Copy link

codecov bot commented Dec 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.99%. Comparing base (a71cfc0) to head (21ac71b).
Report is 16 commits behind head on dev.

Additional details and impacted files
@@             Coverage Diff              @@
##                dev    #1093      +/-   ##
============================================
+ Coverage     74.42%   76.99%   +2.57%     
- Complexity     1227     1691     +464     
============================================
  Files           399      574     +175     
  Lines          5524     7780    +2256     
  Branches        440      525      +85     
============================================
+ Hits           4111     5990    +1879     
- Misses         1220     1547     +327     
- Partials        193      243      +50     
Flag Coverage Δ
integrationTest 72.59% <100.00%> (+3.20%) ⬆️
unitTest 30.11% <0.00%> (-5.86%) ⬇️

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.

wonies
wonies previously approved these changes Dec 31, 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.

로직 바꾸시느라 고생많으셨습니다!

private MockMvc mockMvc;

@Autowired
EntityManager em;
Copy link
Contributor

Choose a reason for hiding this comment

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

em을 쓰는 부분이 없는데, 이건 그냥 선언해도 되는 건가요? 궁금해서 여쭤봅니닷

Page<PublicSchedule> publicSchedules = publicScheduleAdminRepository.findAllByClassification(
detailClassification, pageable);

List<TotalScheduleAdminResDto> publicScheduleList = publicSchedules.stream()
Copy link
Contributor

Choose a reason for hiding this comment

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

자바 11버전 collects

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.

🔨 [Refactoring] Admin 공유일정 조회 API -> 분류별 일정조회 API로 변경
2 participants