Skip to content

Conversation

@kwonjin2
Copy link
Collaborator

#️⃣연관된 이슈

#61

📝작업 내용

  • 롤링 페이퍼 목록 페이지 구현
  • API 데이터를 불러와 인기/최근 롤링 페이퍼 정렬 및 렌더링
  • 캐러셀 기능 추가 (pc: 버튼으로 슬라이스 기능, 태블릿 이하:좌우 스크롤)
  • 스켈레톤 UI 추가 (로딩 상태)
  • 카드 클릭 시 상세 페이지 이동 처리
  • "나도 만들어보기" 버튼 추가 (새 글 작성 페이지 이동)

스크린샷 (선택)

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

- ListPage 구현했습니다. 상세 요구사항은 노션 요구사항 리스트에 작성해뒀습니다.
- 카드 리스트 동적 추가, 반응형, 정렬기능 모두 구현했습니다.
Copy link
Collaborator

@jeonghwanJay jeonghwanJay left a comment

Choose a reason for hiding this comment

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

저는 현재는 수정 할 사항이 없어보입니다.!
테스트 해보고 만약 수정 사항이 있다면 말씀드리겠습니다.
너무 고생하셨습니다 !

Copy link
Collaborator

@zeon0xx0 zeon0xx0 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@vercel
Copy link

vercel bot commented Mar 18, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
rolling-paper ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 18, 2025 4:16am

@kwonjin2 kwonjin2 merged commit d834e4b into main Mar 18, 2025
4 checks passed
Comment on lines +22 to +23
let extraCount = totalUsers - maxVisible;
if (extraCount < 0) extraCount = 0;

Choose a reason for hiding this comment

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

const extraCount = Math.max(totalUsers - maxVisibile, 0);

<div className={styles.reactionsContainer}>
{badges.map((badge, index) => (
<div key={index} {...badge} />
<div key={index} {...badge} className={styles.badge}>

Choose a reason for hiding this comment

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

<div key={index} className={styles.badge} {...badge}>

// 인기 섹션 좌우 이동 핸들러
const handlePopularPrev = () => {
if (popularIndex > 0) {
setPopularIndex(popularIndex - 1);

Choose a reason for hiding this comment

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

익명 화살표 함수를 넘겨 줘서 처리, updater function
setPopularIndex(prev => prev - 1)

profileSection={dynamicProfileImages}
totalUsers={recipient.messageCount}
message={`To. ${recipient.name}`}
userMessage={`명이 작성했어요!`}

Choose a reason for hiding this comment

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

userMessage='명이 작성했어요!'

Comment on lines +241 to +246
const colorMapping = {
beige: "#FFE2AD",
purple: "#ECD9FF",
blue: "#B1E4FF",
green: "#D0F5C3",
};

Choose a reason for hiding this comment

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

1(COLOR_TEXT):1(BACKGROUND_COLOR)
const COLOR_TEXT_BACKGROUND_COLOR_MAP = {

}

Comment on lines +248 to +250
const getHexColor = (apiColor) => {
return colorMapping[apiColor] || "#FFE2AD";
};

Choose a reason for hiding this comment

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

const getHexColor = (colorText) => {
  return colorMapping[colorText] || "#FFE2AD";
};

};

const getHexColor = (apiColor) => {
return colorMapping[apiColor] || "#FFE2AD";

Choose a reason for hiding this comment

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

return colorMapping[apiColor] || DEFAULT_COLOR;

Choose a reason for hiding this comment

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

|| -> ??

Comment on lines +252 to +257
const fetchRecipients = async () => {
const res = await fetch("https://rolling-api.vercel.app/14-6/recipients/");
if (!res.ok) throw new Error("데이터 불러오기 실패");
const data = await res.json();
return data.results || [];
};

Choose a reason for hiding this comment

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

api 계층은 별도 폴더에서 작성해주신 게 좋습니다

Choose a reason for hiding this comment

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

폴더명
src/apis/fetchRecipients.js
src/remotes/fetchRecipients.js

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants