-
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: 어드민 혜택 상점 및 자동완성 검색기능 추가 #906
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.
수고하셨습니다~ 코멘트 몇개만 확인해주세요!
src/main/java/in/koreatech/koin/admin/benefit/repository/AdminBenefitCategoryMapRepository.java
Show resolved
Hide resolved
src/main/java/in/koreatech/koin/admin/benefit/service/AdminBenefitService.java
Outdated
Show resolved
Hide resolved
src/main/java/in/koreatech/koin/admin/benefit/service/AdminBenefitService.java
Outdated
Show resolved
Hide resolved
src/main/java/in/koreatech/koin/admin/benefit/service/AdminBenefitService.java
Show resolved
Hide resolved
src/main/java/in/koreatech/koin/admin/benefit/service/AdminBenefitService.java
Outdated
Show resolved
Hide resolved
src/main/java/in/koreatech/koin/admin/benefit/service/AdminBenefitService.java
Outdated
Show resolved
Hide resolved
src/main/java/in/koreatech/koin/domain/benefit/model/BenefitCategory.java
Show resolved
Hide resolved
src/main/java/in/koreatech/koin/admin/benefit/dto/AdminCreateBenefitShopsResponse.java
Outdated
Show resolved
Hide resolved
src/main/java/in/koreatech/koin/admin/benefit/dto/AdminBenefitShopsResponse.java
Outdated
Show resolved
Hide resolved
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.
크리티컬하진 않은 문제들이라 �Approve 드립니다.
코멘트 확인 부탁드릴게요
String offImageUrl | ||
) { | ||
|
||
public static InnerBenefitResponse from(BenefitCategory benefitCategory) { |
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.
C
외부에서 InnerBenefitResponse 를 만들어서 사용할 일이 있나요?
없다면 접근제어자를 private으로 설정하는건 어떨까요?
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.
잊고있었네요. 좋은 지적 감사합니다! 아래도 동일하게 수정하겠습니다!
String name | ||
) { | ||
|
||
public static InnerShopResponse from(Shop shop) { |
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.
C
외부에서 InnerBenefitResponse 를 만들어서 사용할 일이 있나요?
없다면 접근제어자를 private으로 설정하는건 어떨까요?
ditto
|
||
void deleteById(Integer id); | ||
|
||
int count();; |
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.
R
세미콜론 두개👀
🔥 연관 이슈
🚀 작업 내용
💬 리뷰 중점사항
Normal 버전에 on/off image_url이 없어서 추가하였고, json 필드를 snake_case로 통일했습니다.