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] #74 - 마이페이지, 상점 정보 조회 api 구현 #77

Merged
merged 20 commits into from
Jan 21, 2025

Conversation

sae2say
Copy link
Collaborator

@sae2say sae2say commented Jan 19, 2025

Related issue 🛠

Work Description ✏️

마이페이지 정보를 조회합니다.
상점 id를 받아 특정 상점 정보를 조회합니다.

Trouble Shooting ⚽️

Related ScreenShot 📷

image

image

Uncompleted Tasks 😅

To Reviewers 📢

@hyerinhwang-sailin
Copy link
Collaborator

다른 pr과 겹치는 코멘트들이 많네요 반영 부탁드립니다~
계층 분리가 아직 어색하시겠지만 계속 작업하다 보면 익숙해집니다..ㅎㅎ 화이팅!

@CurrentMember final Long currentStoreId
) {

List<GenrePreference> genreList = storeService.getGenrePreferenceList(OnwerId);
Copy link
Collaborator

Choose a reason for hiding this comment

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

GenrePreference는 storeGenreFacade를 통해 조회해오면 될 것 같습니다!

Copy link
Collaborator Author

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
Copy link
Collaborator

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라고 붙여서 구분해줄게요!

Comment on lines 8 to 9
import org.springframework.stereotype.Repository;

Copy link
Collaborator

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;
Copy link
Collaborator

Choose a reason for hiding this comment

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

안 사용되는 import 문 같은데 맞다면 삭제해주세요!

Copy link
Collaborator

@hyerinhwang-sailin hyerinhwang-sailin left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 👍

@hyerinhwang-sailin hyerinhwang-sailin merged commit 46125be into develop Jan 21, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants