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: 상점 혜택 상세보기 필드 추가 #1123

Merged
merged 3 commits into from
Dec 7, 2024

Conversation

BaeJinho4028
Copy link
Collaborator

🔥 연관 이슈

🚀 작업 내용

  1. 상점 혜택 상세보기 필드 추가

💬 리뷰 중점사항

api는 기존에 있었고, 기능명세서대로 상점 혜택 상세보기 필드 추가하였습니다.

@BaeJinho4028 BaeJinho4028 added 기능 새로운 기능을 개발합니다. Team Business 비즈니스 팀에서 작업할 이슈입니다 labels Dec 6, 2024
@BaeJinho4028 BaeJinho4028 self-assigned this Dec 6, 2024
Copy link

github-actions bot commented Dec 6, 2024

Unit Test Results

343 tests   342 ✔️  1m 30s ⏱️
  41 suites      1 💤
  41 files        0

Results for commit 85c45f4.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@20HyeonsuLee 20HyeonsuLee left a comment

Choose a reason for hiding this comment

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

혜택 페이지에서는 선택한 카테고리에 대한 혜택 설명만 나오도록 되어있습니다. 수정 부탁드려요!

@@ -183,7 +183,8 @@ void setup() {
"is_event": false,
"is_open": true,
"average_rate": 5.0,
"review_count": 1
"review_count": 1,
"benefit_details": ["무료"]
Copy link
Contributor

Choose a reason for hiding this comment

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

혜택 페이지에서는 선택한 혜택 카테고리에 맞는 설명만 나오도록 되어있습니다.
배열 대신 선택한 혜택에 해당하는 설명 하나만 반환하면 될것 같습니다. 관련된 로직도 수정해주세요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

혜택에 해당하는 설명 하나만 반환하도록 수정하였습니다.

Copy link
Contributor

@krSeonghyeon krSeonghyeon left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

@BaeJinho4028 BaeJinho4028 merged commit d874940 into develop Dec 7, 2024
4 checks passed
@BaeJinho4028 BaeJinho4028 deleted the feature/1122-shop-benefit branch December 7, 2024 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team Business 비즈니스 팀에서 작업할 이슈입니다 기능 새로운 기능을 개발합니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

혜택 상세 페이지 api 를 만든다.
3 participants