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: 꿀조합 랭킹 컴포넌트 추가 #478

Merged
merged 10 commits into from
Aug 17, 2023
Merged

Conversation

xodms0309
Copy link
Collaborator

@xodms0309 xodms0309 commented Aug 16, 2023

Issue

✨ 구현한 기능

2023-08-16.8.34.34.mov
  • 캐러셀 컴포넌트 구현
  • 꿀조합 랭킹 리스트 컴포넌트 추가

📢 논의하고 싶은 내용

  • Carousel 컴포넌트에 있는 extendedCarouselList가 첫 번째 있는 아이템을 뒤에 붙이는 눈속임같은 건데요 이 눈속임 때문에 얘는 좀 더 오랫동안 노출됩니다. 1등에 대한 혜택이라고 하죠 뭐..
  • 추가적으로 현재 꿀조합 랭킹 api가 꿀조합 목록 조회의 api와 다른데 (사용된 상품, 날짜가 없음) api 변경을 할까요 아니면 컴포넌트를 새로 만들까요? 현재는 RecipeItem 컴포넌트 재활용 하고 있습니다.
  • 이 컴포넌트는 그럼 꿀조합 페이지에 들어가나요? 아니면 메인 페이지?

🎸 기타

x

⏰ 일정

  • 추정 시간 : 1시간
  • 걸린 시간 : 3시간

@github-actions
Copy link

github-actions bot commented Aug 16, 2023

Unit Test Results

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

Results for commit 0f96aaf.

♻️ This comment has been updated with latest results.

@hae-on
Copy link
Collaborator

hae-on commented Aug 16, 2023

추가적으로 현재 꿀조합 랭킹 api가 꿀조합 목록 조회의 api와 다른데 (사용된 상품, 날짜가 없음) api 변경을 할까요 아니면 컴포넌트를 새로 만들까요? 현재는 RecipeItem 컴포넌트 재활용 하고 있습니다.

지금 캡쳐 모습대로 캐러셀이 들어가는게 아니라 상품, 날짜 없이 들어간다는 말이죠?? RecipeItem이 아마 제가 member에서 사용한다고 좀 누더기일텐데 거기서 더 진행했다가 더 더러워질 거 같으면 저는 새로 파는걸 더 추천드립니다!

이 컴포넌트는 그럼 꿀조합 페이지에 들어가나요? 아니면 메인 페이지?

아마 메인페이지에 자리 잡게 될 거 같아요! 내일 같이 화면 보면서 어디에 배치해야 할 지 더 말해봐요~

@Leejin-Yang
Copy link
Collaborator

  • 이 컴포넌트는 그럼 꿀조합 페이지에 들어가나요? 아니면 메인 페이지?

현재 메인 페이지에 요소가 너무 많아서 꿀조합 페이지에서 상단에 가장 인기 목록(랭킹) 보여주고 아래에 리스트를 보여주자는 이야기가 있었습니다! 내일 더 이야기해봐요.

  • 추가적으로 현재 꿀조합 랭킹 api가 꿀조합 목록 조회의 api와 다른데 (사용된 상품, 날짜가 없음) api 변경을 할까요 아니면 컴포넌트를 새로 만들까요? 현재는 RecipeItem 컴포넌트 재활용 하고 있습니다.

UI에 따라 달라질거 같아요. 지금 노션에 작성된 랭킹 API는 회의 때 이후로 업데이트 되어 있지 않아서, UI에 따라 업데이트 해야할 거 같습니다(레시피 목록에서도 없었다가 날짜와 상품 추가했습니다). 만약 날짜나 상품이 필요 없다면 새로 만들어서 해도 좋아보입니다. API 관련 사항은 백엔드랑 이야기 해보시죠.

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.

캐러셀 굿입니다 👍

Comment on lines 6 to 9
carouselList: {
id: number;
children: ReactNode;
}[];
Copy link
Collaborator

Choose a reason for hiding this comment

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

따로 타입으로 빼는게 좋아보여요!

@xodms0309
Copy link
Collaborator Author

@hae-on @Leejin-Yang
스크린샷 2023-08-17 오후 2 15 39

  • 컴포넌트 ui가 변경되었습니다.

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.

확시랗빕다

@xodms0309 xodms0309 merged commit b1d78ec into develop Aug 17, 2023
3 checks passed
@xodms0309 xodms0309 deleted the feat/issue-425 branch August 17, 2023 06:38
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