Skip to content

[Feat/#192]: 책정보 없는경우 대체 ui 적용#193

Merged
SebellKo merged 10 commits intodevfrom
feat/#192
Aug 4, 2025
Merged

[Feat/#192]: 책정보 없는경우 대체 ui 적용#193
SebellKo merged 10 commits intodevfrom
feat/#192

Conversation

@SebellKo
Copy link
Contributor

@SebellKo SebellKo commented Aug 4, 2025

📍 작업 내용

네이버 API 오류로 인해 책 정보가 없는경우 대체 ui를 적용했습니다.

  • 프로필 피드 아이템
  • 도서 상세 페이지
  • 메모
  • 리뷰
  • 우물 아이템
  • 피드

📍 구현 결과 (선택)

스크린샷 2025-08-04 오후 10 55 38 스크린샷 2025-08-04 오후 10 57 18 스크린샷 2025-08-04 오후 10 59 33

📍 기타 사항

리뷰어가 특별히 봐주었으면 하는 부분이나 주의사항, 알림사항 등이 있다면 작성해주세요.

ex) 더미 데이터를 넣어서 기능 구현한 상태입니다!


@vercel
Copy link

vercel bot commented Aug 4, 2025

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

Name Status Preview Comments Updated (UTC)
frolog ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 4, 2025 2:25pm

@SebellKo SebellKo requested a review from JIMIN1020 August 4, 2025 14:02
@SebellKo SebellKo self-assigned this Aug 4, 2025
@SebellKo SebellKo added the feat 기능 구현 label Aug 4, 2025
/** 도서 상세 페이지, 리뷰/메모 리스트 내 도서 정보 컴포넌트 (도서 커버, 배경 포함) */
function BookInfo({ bookId, bookData, canClick = false }: Props) {
const isInFeed = useIsInFeed();
const isEmpty = !bookData.title;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

author가 unknown으로 렌더링 되는 경우도 있어서 title이 없는 경우를 empty 상태로 판단했습니다.


if (!bookData) return null;

const fallbackCategory = 'cartoon';
Copy link
Contributor Author

Choose a reason for hiding this comment

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

디자인 시안상 fallback ui 색상이 cartoon이라 이걸로 적용했습니다.

const height = page > 400 ? page * 0.15 : 55;
const isReading = status === 'reading';
const hasMemo = memo_cnt > 0;
const fallbackCategory = 'economic_business';
Copy link
Contributor Author

Choose a reason for hiding this comment

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

여기도 마찬가지로 디자인 시안상 fallback ui 카테고리가 이거여서 이걸로 설정했습니다.

Copy link
Contributor

@JIMIN1020 JIMIN1020 left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다~

@SebellKo SebellKo merged commit 9fdade1 into dev Aug 4, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants