[WTH-476] 어드민용 일반 일정 조회 api 구현 - #100
Hidden character warning
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 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 |
hyxklee
left a comment
hyxklee
left a comment
There was a problem hiding this comment.
메스드명만 한 번 고민해주시고 완료되면 머지해주세용
이렇게 API가 추가되는 경우는 문제 없지만, 기존에 쓰는 API가 있다면 하위호환도 고려해야하니 그런 부분도 인지하고 백엔드 개발을 하면 좋습니당ㅇ구리
| return (events + sessions).sortedBy { it.start } | ||
| } | ||
|
|
||
| fun findAdminEvents( |
There was a problem hiding this comment.
메서드 명이 AdminEvents라면 약간 어드민용 이벤트가 있나?
라는 생각이 들 것 같아용 findEventsByAdmin은 어떤가용가리
There was a problem hiding this comment.
말씀해 주신대로 findEventsByAdmin으로 수정했습니다!
| start: LocalDateTime, | ||
| end: LocalDateTime, | ||
| ): List<EventResponse> { | ||
| clubMemberPolicy.getActiveMember(clubId, userId) |
There was a problem hiding this comment.
jwt 필터에서는 유저 역할만 체크하고 있어서 서비스 레이어에서 clubPermissionPolicy.requireAdmin()으로 어드민 권한을 검증하도록 수정했습니닷...!
2 participants
📌 Summary
어드민 일정 관리 페이지의 일반 일정 목록 조회 API를 추가합니다.
📝 Changes
What
Why
How
EventAdminController에 GET 엔드포인트 추가GetScheduleQueryService에findAdminEvents메서드 추가 (cardinal 유무에 따라 기수 필터링 또는 전체 조회)📸 Screenshots / Logs
💡 Reviewer 참고사항
✅ Checklist