-
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: ProductItem, ProductList 컴포넌트 추가 #43
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.
황펭 수고 했어요~ 리뷰 몇 개 달아놨습니다
근데 typeRoots에 저거 추가하면 뭐가 바뀌나요??
<Text size="lg" weight="bold"> | ||
{name} | ||
</Text> | ||
<Text color={theme.textColors.info}>{price.toLocaleString('ko-KR')}원</Text> |
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.
시안을 보면 가격이 사이즈가 xs
여야 할 것 같아요
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.
xs 사이즈 너무 작아보여서 일단 뒀습니다..
@@ -0,0 +1,29 @@ | |||
import productImage from './mock_img.jpg'; |
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.
아직 msw 안 했어도 이 파일은 mocks/data 아래에 있어도 될 것 같아요
export interface Product { | ||
id: number; | ||
name: string; | ||
price: number; | ||
image: string; | ||
averageRating: number; | ||
reviewCount: number; | ||
} |
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.
이 타입도 types아래에 product.ts 이런식으로 만들면 좋을 것 같아요
export default meta; | ||
type Story = StoryObj<typeof ProductList>; |
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.
따로 수정할 사항은 없습니다! 수고하셨어요~~
@xodms0309 @hae-on
확인해주세요~ |
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.
많은 일이 있었던 그의 브랜치.. 수고하셨습니다~
Issue
✨ 구현한 기능
📢 논의하고 싶은 내용
🎸 기타