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

Feat: 탐색 페이지 퍼블리싱, 데이터 바인딩 (임시) #25

Merged
merged 22 commits into from
Feb 11, 2024

Conversation

Nahyun-Kang
Copy link
Contributor

개요

  • 개요는 변경 사항 및 관련 이슈에 대해 간단하게 작성해주세요.
  • 해당 PR에 대한 리뷰어와 라벨을 생성해 주세요.

작업 사항

  • 탐색 페이지 랜딩 페이지로 작업
  • 퍼블리싱 및 데이터 바인딩 (일부)
  • @kanglocal 현지님 풀 받으셔서 합체 해주시고 다시 올려주시면 합쳐보겠습니다! 아무래도 simpleList 컴포넌트 밖으로 빼야할 것 같아서 공용으로 빼주실 수 있으신가요...?👀

참고 사항 (optional)

  • 현지님께서 제작하신 컴포넌트와 합치기 위해 임시로 올린 PR입니다.
  • 현지님께서 합쳐주시고 저는 그 PR을 다시 풀 받아오고 코드 정리를 할 예정입니다. 참고 부탁드립니다!🙇‍♀️

관련 이슈 (optional)


스크린샷


리뷰어에게

  • 컴포넌트 합체 전이니 코드리뷰는 이후 정식 PR에서 부탁드립니다!

Copy link

vercel bot commented Feb 11, 2024

@Nahyun-Kang is attempting to deploy a commit to the Eujin Ahn's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@ParkSohyunee ParkSohyunee left a comment

Choose a reason for hiding this comment

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

나현님, 연휴동안에도 많은 부분 작업하시느라 정말 고생많으셨습니다!!👍
요청 주신대로 추후 PR에서 꼼꼼히 다시 확인하도록 하겠습니다. 감사합니다.🥰

Comment on lines +58 to +66
export interface ListItemsType {
id: number;
rank: number;
title: string;
comment: string;
link: string;
imageUrl: string;
}

Copy link
Contributor

Choose a reason for hiding this comment

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

나현님 이부분만 나중에 합친 PR올리실 때, 기존에 있는 타입(ItemType)을 사용하면 될 것 같습니다!
+. 나현님 아이템 부분이랑 피드페이지 아이템 부분 켭치는 부분이 있어서 타입 작업하실때 타입 같이 보면 좋을 것 같습니당~!!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

앗 그렇군요..! 생각해보니 겹치는 타입이 많을 것 같네용...👀 이후에 PR 합칠 때 기존에 만들어두신 타입 사용해보겠습니당!!🤩

@Nahyun-Kang Nahyun-Kang added the Feat 구현 label Feb 11, 2024
@Nahyun-Kang Nahyun-Kang added the 추후 확인 필요 나중에 다시 살펴봐야할 PR (미해결 문제 존재, 급한 머지 등) label Feb 11, 2024
@Nahyun-Kang Nahyun-Kang changed the title Feat: 탐색 페이지 퍼블리싱, 데이터 바인딩 Feat: 탐색 페이지 퍼블리싱, 데이터 바인딩 (임시) Feb 11, 2024
@Nahyun-Kang Nahyun-Kang merged commit 62cc789 into 8-Sprinters:dev Feb 11, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 구현 추후 확인 필요 나중에 다시 살펴봐야할 PR (미해결 문제 존재, 급한 머지 등)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants