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

feat: 혜택 카테고리, 상점 조회 API 작성 #887

Merged
merged 17 commits into from
Sep 18, 2024

Conversation

20HyeonsuLee
Copy link
Contributor

@20HyeonsuLee 20HyeonsuLee commented Sep 14, 2024

🔥 연관 이슈

🚀 작업 내용

  1. 혜택 카테고리 조회 API를 작성했습니다.
  2. 특정 혜택을 제공하는 상점을 조회하는 API를 작성했습니다.
  3. dto, repository, entity등 하나의 패키지에 클래스가 너무 많은 경우가 있어서 정리 했습니다.

💬 리뷰 중점사항

이번 PR머지시키고 상점 조회에서 query dsl활용해서 성능 최적화 한번 시도해보려고 합니다.

@20HyeonsuLee 20HyeonsuLee self-assigned this Sep 14, 2024
@github-actions github-actions bot added the 기능 새로운 기능을 개발합니다. label Sep 14, 2024
Copy link

github-actions bot commented Sep 14, 2024

Unit Test Results

291 tests   290 ✔️  1m 15s ⏱️
  33 suites      1 💤
  33 files        0

Results for commit 458e58f.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@BaeJinho4028 BaeJinho4028 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다. 간단한거 몇가지만 체크해주세요.
(저도 전에 1000줄 pr날렸다가 팀원들과 멘토님께 피드백 받고, 구조 리팩토링도 따로 pr 나누려고 하는데 한번 고민해보시길)

Copy link
Contributor

@dradnats1012 dradnats1012 left a comment

Choose a reason for hiding this comment

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

파일체인지 보고 엄두도 안났는데 패키지 변경이었네요 👍


private final ShopBenefitService shopBenefitService;

@GetMapping("/benefit/categories")
Copy link
Contributor

Choose a reason for hiding this comment

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

/benefit 공통이라 뺴도 될것 같아요

public record BenefitCategoryResponse(
@Schema(description = "혜택 카테고리 리스트", requiredMode = NOT_REQUIRED)
List<InnerBenefitResponse> benefits

Copy link
Contributor

Choose a reason for hiding this comment

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

개행 발견

@20HyeonsuLee 20HyeonsuLee merged commit 2cf493c into develop Sep 18, 2024
4 checks passed
@20HyeonsuLee 20HyeonsuLee deleted the feature/880-get-benefit-category branch September 18, 2024 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
기능 새로운 기능을 개발합니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

혜택에 해당하는 상점을 조회하는 API 작성 혜택 카테고리 조회 API작성
3 participants