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

[FE] feat: 리뷰 상세 페이지 구현 #762

Merged
merged 12 commits into from
Oct 16, 2023
Merged

[FE] feat: 리뷰 상세 페이지 구현 #762

merged 12 commits into from
Oct 16, 2023

Conversation

hae-on
Copy link
Collaborator

@hae-on hae-on commented Oct 13, 2023

Issue

✨ 구현한 기능

리뷰 상세 페이지를 구현하였습니다.
상품 제목이 링크라는 것을 표현해달라는 의견을 받아 링크 아이콘으로 대체하였습니다.

스크린샷 2023-10-14 오전 12 39 14

기존 마이페이지의 디자인을 시안과 같이 수정하였습니다.

스크린샷 2023-10-14 오전 12 38 57

로딩 시계를 시계 방향으로 바꾸었습니다.

북마크 로직을 삭제하였습니다.

📢 논의하고 싶은 내용

  • 논의하고 싶은 내용을 작성합니다.

review detail에 있는 리뷰 내용이 좋아요 버튼 빼고 사실 기존 리뷰와 똑같아요.
복붙했는데 솔직히 컴포넌트를 재사용할까 하다가 또 좋아요 로직에 대해 분기가 나뉠 거 같아 그냥 뒀습니다.
컴포넌트가 많아질수록 고민이 많아지네요 🧐

type에 대해 궁금한게 있는데요.
categoryType은 무조건 'food'와 'store' 아닌가요?
기존 코드에서 긁어왔는데 string으로 되어있더라구요.
'food'랑 'store'로 바꿀까요?? 따로 'food'|'store'라는 타입을 본 거 같은데,,,

🎸 기타

  • 특이 사항이 있으면 작성합니다.

⏰ 일정

  • 추정 시간 : 2시간
  • 걸린 시간 : 2시간

@github-actions
Copy link

github-actions bot commented Oct 13, 2023

Test Results

2 tests   2 ✔️  0s ⏱️
1 suites  0 💤
1 files    0

Results for commit 38b1fe6.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@Leejin-Yang Leejin-Yang left a comment

Choose a reason for hiding this comment

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

해온 수고했어요! 수정할건 보이지 않네요. 완벽합니다!

review detail에 있는 리뷰 내용이 좋아요 버튼 빼고 사실 기존 리뷰와 똑같아요.

맞네요. UI가 똑같네요.! 내부 로직이라던가, 데이터가 달라서 이런 딜레마가 발생하는군요. 생각해보니 다른 컴포넌트에서도 그랬네요. 아니면 스타일드 코드를 재사용해서 컴포넌트에서 import 하는건 어떻게 생각하나요?? 대규모 공사가 되겠지만... 우선은 따로 작성한 것은 좋아보입니다!

'food'랑 'store'로 바꿀까요?? 따로 'food'|'store'라는 타입을 본 거 같은데,,,

저도 어디서 본 거 같은데... 찾아봤는데 types/common에 CategoryVariant가 있네요!

새 페이지, 마이페이지 스타일 수정, 시계방향으로 수정, 북마크 로직 삭제까지 고생했습니다 👍

@@ -40,7 +40,7 @@ const rotate = keyframes`
}

100% {
transform: rotate(-360deg);
transform: rotate(360deg);
Copy link
Collaborator

Choose a reason for hiding this comment

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

⏰👍

@hae-on
Copy link
Collaborator Author

hae-on commented Oct 14, 2023

@Leejin-Yang

스타일드 코드를 재사용해서 컴포넌트에서 import 하는건 어떻게 생각하나요??

그러면 해당 스타일 코드를 따로 파일로 분리하고 그걸 import한다는 말이 맞을까요??
전 좋다고 생각합니다!
그렇다면 그 분리된 스타일 파일들은 어디로 가야하며 이름은 어떻게 짓는게 좋을까요??

@Leejin-Yang
Copy link
Collaborator

@hae-on

옙 맞아요! UI가 같아서 스타일은 공통적으로 사용하니 중복된 코드를 제거 할 수 있겠다고 생각했어요.
components 폴더에 도메인 별로 분리되어 있으니 그 안에 스타일 파일을 만들어도 될 거 같고(components/Review/styles.ts)
아니면 styles 폴더에 도메인 별로 폴더를 만들어서 작성해도 될 거 같네요.!

@hae-on
Copy link
Collaborator Author

hae-on commented Oct 14, 2023

@Leejin-Yang
styles 폴더에 해보려고 했는데 해당 파일 이름 짓기가 애매해서 일단 components/Review/styles.ts 이렇게 만들어보겠습니다.
그런데 import해서 쓰려면 import * as S from './ReviewItem.styles'; 이런식으로 해야 하나요...?
저걸 안하려면 하나하나 다 import 해야하는거 같은데...? 😬

@Leejin-Yang
Copy link
Collaborator

Leejin-Yang commented Oct 14, 2023

@hae-on

흠.. 그러게요.. import 문이 길어지는게 문제라면 해온 말처럼 * 로 가져올 수 있겠네요..
기존 컴포넌트나 훅이나 index에서 필요한 것만 import 해오기 때문에 (import { Skeleton, SvgIcon } from '@/components/Common';) 사용하는 스타일 컴포넌트만 import 해도 괜찮아 보입니다!

만약 한다면 이 작업은 따로 브랜치 파서 작업해도 될거 같네요.!

@hae-on
Copy link
Collaborator Author

hae-on commented Oct 14, 2023

@Leejin-Yang @xodms0309
갑자기 생각났는데 베스트 리뷰가 생겼잖아요?
이것도 클릭하면 상세 리뷰 페이지로 이동시켜야 할까요?

이게 베스트 리뷰 말줄임표 없이 바로 다 보여줄것인지 이동시킬 것인지
다같이 이야기해보자 해놓고 까먹었네요...
어떻게 하는게 좋을 거 같나요??

@Leejin-Yang
Copy link
Collaborator

@hae-on

제 생각에는 로그인 유도하는 미리보기여서 베스트 리뷰에서는 안 가도 될거 같네요.!

Copy link
Collaborator

@xodms0309 xodms0309 left a comment

Choose a reason for hiding this comment

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

해온 수고했어요! 리뷰 확인해주세요~
추가로 지금 ReviewRankingItem에서 memberPage에 대한 분기 처리가 이루어지고 있는데요 리뷰 삭제가 들어가게 된다면 추가적인 로직이 들어가기 때문에 아예 다른 컴포넌트로 처리해야될 것 같아요! 이 부분은 제가 리뷰 삭제 구현하면서 하겠습니다~~

Comment on lines -9 to +10
bookmark?: boolean;
link?: string;
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

Comment on lines +56 to +58
const ProductName = styled(Heading)`
margin: 0 5px 0 16px;
`;
Copy link
Collaborator

Choose a reason for hiding this comment

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

위에 css={{ marginLeft: '20px' }} 속성이 있는데 여기는 또 16px이네요?! 어떤 것이 적용되어야 하나요??

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

아마 지금 css가 적용이 안 돼서 margin: 0 5px 0 16px; 얘가 적용될 거에요! css={{ marginLeft: '20px' }}는 삭제처리 했습니다.

import type { ReviewDetailResponse } from '@/types/response';

const fetchReviewDetail = async () => {
const response = await reviewApi.get({ params: '/reviews' });
Copy link
Collaborator

Choose a reason for hiding this comment

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

params에 reviews가 아닌 reviewId가 들어가야 할 것 같네요!

@hae-on
Copy link
Collaborator Author

hae-on commented Oct 15, 2023

@xodms0309

리뷰 반영 했습니다! 확인해주세요~~

Copy link
Collaborator

@xodms0309 xodms0309 left a comment

Choose a reason for hiding this comment

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

굿! 코멘트 하나만 확인해주시고 머지 해주세용 🌞

Comment on lines 12 to 14
const useReviewDetailQuery = (reviewId: number) => {
return useSuspendedQuery(['review'], () => fetchReviewDetail(reviewId));
};
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기 쿼리키에 reviewId도 들어가야 할 것 같아요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

얍! product detail처럼 'detail' key도 추가했습니다~

@hae-on hae-on merged commit 215003f into develop Oct 16, 2023
3 checks passed
@hae-on hae-on deleted the feat/issue-746 branch October 16, 2023 03:47
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.

[FE] 리뷰 상세 페이지 구현
3 participants