-
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] #74 - 마이페이지, 상점 정보 조회 api 구현 #77
Conversation
src/main/java/com/napzak/domain/store/api/StoreGenreFacade.java
Outdated
Show resolved
Hide resolved
src/main/java/com/napzak/domain/store/api/controller/StoreController.java
Outdated
Show resolved
Hide resolved
src/main/java/com/napzak/domain/store/api/controller/StoreController.java
Outdated
Show resolved
Hide resolved
src/main/java/com/napzak/domain/store/api/controller/StoreController.java
Outdated
Show resolved
Hide resolved
src/main/java/com/napzak/global/common/config/SecurityConfig.java
Outdated
Show resolved
Hide resolved
src/main/java/com/napzak/domain/store/api/service/StoreService.java
Outdated
Show resolved
Hide resolved
src/main/java/com/napzak/domain/store/api/service/StoreService.java
Outdated
Show resolved
Hide resolved
다른 pr과 겹치는 코멘트들이 많네요 반영 부탁드립니다~ |
…er storeId -> currentStoreId)
@CurrentMember final Long currentStoreId | ||
) { | ||
|
||
List<GenrePreference> genreList = storeService.getGenrePreferenceList(OnwerId); |
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.
GenrePreference는 storeGenreFacade를 통해 조회해오면 될 것 같습니다!
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.
genrePreference가 store 도메인 안에 있어서 storeService로 호출하였습니다!
String storeDescription, | ||
String storePhoto, | ||
String storeCover, | ||
List<GenrePreferenceResponse> genrePreferences |
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.
GenrePreferenceResponse -> GenrePreferenceDto
Response는 controller에서 최종적으로 반환하는 dto에만 붙이고, response 내에서만 사용되는 건 뒤에 Dto라고 붙여서 구분해줄게요!
import org.springframework.stereotype.Repository; | ||
|
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.
안 사용되는 import 문 같은데 맞다면 삭제해주세요!
@@ -1,6 +1,7 @@ | |||
package com.napzak.domain.store.core; | |||
|
|||
import java.util.List; | |||
import java.util.stream.Collectors; |
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.
안 사용되는 import 문 같은데 맞다면 삭제해주세요!
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.
고생하셨습니다 👍
Related issue 🛠
Work Description ✏️
마이페이지 정보를 조회합니다.
상점 id를 받아 특정 상점 정보를 조회합니다.
Trouble Shooting ⚽️
Related ScreenShot 📷
Uncompleted Tasks 😅
To Reviewers 📢