-
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: 어드민 상점 혜택 카테고리 수정 기능 추가 #942
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/service/AdminBenefitService.java
Show resolved
Hide resolved
BenefitCategory benefitCategory = adminBenefitCategoryRepository.getById(categoryId); | ||
benefitCategory.update(request.title(), request.detail(), request.onImageUrl(), request.offImageUrl()); | ||
return AdminModifyBenefitCategoryResponse.from(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
수정할때도 동일한 이름의 카테고리가 있는지 검증해줄 필요가 있어보여요
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.
👍 👍
🔥 연관 이슈
🚀 작업 내용
💬 리뷰 중점사항
명세서대로 어드민 상점 혜택 카테고리 수정 기능 추가 및 테스트 간단하게 작성하였습니다.