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

[refactor] 메인 페이지 조회 시, 캐러셀 리스트를 캐러셀 번호 순으로 정렬하여 응답 #221

Closed
1 task done
hoonyworld opened this issue Oct 1, 2024 · 0 comments · Fixed by #222
Assignees

Comments

@hoonyworld
Copy link
Member

hoonyworld commented Oct 1, 2024

Feature Issue 📌

  • 기존에는 promotionId 순서대로 캐러셀 리스트를 반환하고 있었습니다.
  • 하지만, 관리자 페이지를 제작하면서 캐러셀의 순서를 CarouselNumber 필드로 관리할 필요가 생겼습니다.
  • 이를 위해 캐러셀 번호 순서대로 리스트를 반환하도록 리팩토링을 진행하고자 합니다.

Todo ✔️

  • 메인 페이지 조회 시, 캐러셀 리스트를 캐러셀 번호 순으로 정렬하여 응답
@hoonyworld hoonyworld self-assigned this Oct 1, 2024
hoonyworld added a commit that referenced this issue Oct 1, 2024
…tions 메서드 이동 및 코드 포맷팅 적용
hoonyworld added a commit that referenced this issue Oct 2, 2024
… 최적화 (#222)

* [#221] rename: DTO 이름 변경 및 import문 최신화

* [#221] chore(AdminApi): 통일성을 위해 mapping 어노테이션 삭제

* [#221] rename(HomeFindRequest): 요청 DTO 네이밍 구체화

* [#221] rename(HomeFindAllResponse): 응답 DTO 네이밍 구체화

* [#221] refactor(PerformanceService): getHomePerformanceList, getPromotions 메서드 이동 및 코드 포맷팅 적용

* [#221] feat(HomeService): 공연 및 홍보 조회 서비스 로직 최적화

* [#221] feat(ScheduleService): 공연의 회차 중 가장 dueDate가 적은 회차의 dueDate를 반환하는 메서드 구현

* [#221] refactor(HomePromotionDetail): 정적 팩토리 메서드 내에서 초기화해서 return 하도록 변경

* [#221] refactor(HomePerformanceDetail): 정적 팩토리 메서드 내에서 초기화해서 return 하도록 변경

* [#221] feat(HomeApi): Home API 스웨거 명세서 작성

* [#221] refactor(HomeController): 로직 최적화

* [#221] fix(HomeService): 음수는 내림차순, 양수는 오름차순으로 정렬하도록 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment