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: ProductItem, ProductList 컴포넌트 추가 #43

Merged
merged 21 commits into from
Jul 18, 2023

Conversation

Leejin-Yang
Copy link
Collaborator

@Leejin-Yang Leejin-Yang commented Jul 16, 2023

Issue

✨ 구현한 기능

  • ProductItem 컴포넌트 추가
  • ProductList 컴포넌트 추가
  • image.d.ts 파일 추가

📢 논의하고 싶은 내용

  • 목 데이터 만들면서 이미지 모듈을 못 읽어 image.d.ts 파일을 추가 했습니다. 추가하면서 tsconfig 설정(typeRoots)을 추가했는데 디자인 시스템에도 설정 추가(styled.d.ts)하면 아마 자동완성 할 수 있겠다는 생각이 듭니다.
  • ProductItem 만들면서 ProductList도 만들었습니다.
  • msw 쓰면 목 데이터는 mocks 폴더로 이동할 듯 합니다. 임의로 작성했습니다.
  • 리뷰와 평점에 이모지는 svg 완료되면 대체할 예정입니다.

🎸 기타

스크린샷 2023-07-16 오후 6 45 09 스크린샷 2023-07-16 오후 6 45 18

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.

황펭 수고 했어요~ 리뷰 몇 개 달아놨습니다
근데 typeRoots에 저거 추가하면 뭐가 바뀌나요??

<Text size="lg" weight="bold">
{name}
</Text>
<Text color={theme.textColors.info}>{price.toLocaleString('ko-KR')}원</Text>
Copy link
Collaborator

Choose a reason for hiding this comment

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

시안을 보면 가격이 사이즈가 xs 여야 할 것 같아요

Copy link
Collaborator Author

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

Choose a reason for hiding this comment

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

아직 msw 안 했어도 이 파일은 mocks/data 아래에 있어도 될 것 같아요

Comment on lines 3 to 10
export interface Product {
id: number;
name: string;
price: number;
image: string;
averageRating: number;
reviewCount: number;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 타입도 types아래에 product.ts 이런식으로 만들면 좋을 것 같아요

Comment on lines +10 to +11
export default meta;
type Story = StoryObj<typeof ProductList>;
Copy link
Collaborator

Choose a reason for hiding this comment

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

이거 통일에 대해서 얘기해봅시당

Copy link
Collaborator

@hae-on hae-on left a comment

Choose a reason for hiding this comment

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

따로 수정할 사항은 없습니다! 수고하셨어요~~

@Leejin-Yang
Copy link
Collaborator Author

@xodms0309 @hae-on
여러가지 수정사항이 있었습니다.

  • mocks로 목데이터 이동
  • svg 적용

확인해주세요~

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.

많은 일이 있었던 그의 브랜치.. 수고하셨습니다~

@Leejin-Yang Leejin-Yang merged commit b88e130 into develop Jul 18, 2023
@Leejin-Yang Leejin-Yang deleted the feat/issue-26 branch July 18, 2023 00:52
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