-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: 상점 혜택 상세보기 필드 추가 #1123
Conversation
There was a problem hiding this 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": ["무료"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
혜택 페이지에서는 선택한 혜택 카테고리에 맞는 설명만 나오도록 되어있습니다.
배열 대신 선택한 혜택에 해당하는 설명 하나만 반환하면 될것 같습니다. 관련된 로직도 수정해주세요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
혜택에 해당하는 설명 하나만 반환하도록 수정하였습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다
🔥 연관 이슈
🚀 작업 내용
💬 리뷰 중점사항
api는 기존에 있었고, 기능명세서대로 상점 혜택 상세보기 필드 추가하였습니다.