-
Notifications
You must be signed in to change notification settings - Fork 0
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
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.
해온 수고했어요! 수정할건 보이지 않네요. 완벽합니다!
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); |
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한다는 말이 맞을까요?? |
옙 맞아요! UI가 같아서 스타일은 공통적으로 사용하니 중복된 코드를 제거 할 수 있겠다고 생각했어요. |
@Leejin-Yang |
흠.. 그러게요.. import 문이 길어지는게 문제라면 해온 말처럼 * 로 가져올 수 있겠네요.. 만약 한다면 이 작업은 따로 브랜치 파서 작업해도 될거 같네요.! |
@Leejin-Yang @xodms0309 이게 베스트 리뷰 말줄임표 없이 바로 다 보여줄것인지 이동시킬 것인지 |
제 생각에는 로그인 유도하는 미리보기여서 베스트 리뷰에서는 안 가도 될거 같네요.! |
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.
해온 수고했어요! 리뷰 확인해주세요~
추가로 지금 ReviewRankingItem에서 memberPage에 대한 분기 처리가 이루어지고 있는데요 리뷰 삭제가 들어가게 된다면 추가적인 로직이 들어가기 때문에 아예 다른 컴포넌트로 처리해야될 것 같아요! 이 부분은 제가 리뷰 삭제 구현하면서 하겠습니다~~
bookmark?: boolean; | ||
link?: string; |
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.
👍
const ProductName = styled(Heading)` | ||
margin: 0 5px 0 16px; | ||
`; |
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.
위에 css={{ marginLeft: '20px' }}
속성이 있는데 여기는 또 16px이네요?! 어떤 것이 적용되어야 하나요??
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.
아마 지금 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' }); |
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.
params에 reviews가 아닌 reviewId가 들어가야 할 것 같네요!
리뷰 반영 했습니다! 확인해주세요~~ |
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.
굿! 코멘트 하나만 확인해주시고 머지 해주세용 🌞
const useReviewDetailQuery = (reviewId: number) => { | ||
return useSuspendedQuery(['review'], () => fetchReviewDetail(reviewId)); | ||
}; |
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.
여기 쿼리키에 reviewId도 들어가야 할 것 같아요!
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.
얍! product detail처럼 'detail' key도 추가했습니다~
Issue
✨ 구현한 기능
리뷰 상세 페이지를 구현하였습니다.
상품 제목이 링크라는 것을 표현해달라는 의견을 받아 링크 아이콘으로 대체하였습니다.
기존 마이페이지의 디자인을 시안과 같이 수정하였습니다.
로딩 시계를 시계 방향으로 바꾸었습니다.
북마크 로직을 삭제하였습니다.
📢 논의하고 싶은 내용
review detail에 있는 리뷰 내용이 좋아요 버튼 빼고 사실 기존 리뷰와 똑같아요.
복붙했는데 솔직히 컴포넌트를 재사용할까 하다가 또 좋아요 로직에 대해 분기가 나뉠 거 같아 그냥 뒀습니다.
컴포넌트가 많아질수록 고민이 많아지네요 🧐
type에 대해 궁금한게 있는데요.
categoryType은 무조건 'food'와 'store' 아닌가요?
기존 코드에서 긁어왔는데 string으로 되어있더라구요.
'food'랑 'store'로 바꿀까요?? 따로 'food'|'store'라는 타입을 본 거 같은데,,,
🎸 기타
⏰ 일정