Skip to content

[#77] Refactor/와인상세페이지 와인 카드#87

Merged
qpalkim merged 3 commits intomainfrom
refactor/와인상세페이지_와인카드
Feb 15, 2025

Hidden character warning

The head ref may contain hidden characters: "refactor/\uc640\uc778\uc0c1\uc138\ud398\uc774\uc9c0_\uc640\uc778\uce74\ub4dc"
Merged

[#77] Refactor/와인상세페이지 와인 카드#87
qpalkim merged 3 commits intomainfrom
refactor/와인상세페이지_와인카드

Conversation

@qpalkim
Copy link
Collaborator

@qpalkim qpalkim commented Feb 15, 2025

#️⃣ Issue

📝 Description

와인 상세 페이지 와인 카드 컴포넌트 리팩토링 작업

  • fetchWineDetail.ts 파일을 page.tsx 파일에 불러와 WineContainer 컴포넌트와 ReviewContainer 컴포넌트의 로딩, 에러 처리를 한번에 하도록 함
  • WineContainer 컴포넌트와 ReviewContainer 컴포넌트는 중복된 API 요청 없이 데이터를 넘겨 받아 렌더링함
  • 유효한 와인 id가 아닐 시 NotFound 컴포넌트 렌더링
  • 로그인 상태가 아닐 시 alert로 경고창을 띄운 후, '/signin' 페이지로 이동시킴
  • 데이터 페치 실패 시 refresh 컴포넌트 렌더링
  • 와인 카드 스켈레톤 UI 작업
스켈레톤

스크린샷 2025-02-15 172559
스크린샷 2025-02-15 172644

🌀 PR Type

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항 (오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

✅ Checklist

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • Branch Convention 확인

    feat/ 피쳐, fix/ 버그 수정, refactor/ 개선

  • Base Branch 확인
  • 커밋 메시지 컨벤션 준수
  • 적절한 Label 지정
  • Assignee 및 Reviewer 지정

Test Checklist

  • 로컬 작동 확인

Additional Notes

  • (없음)

@qpalkim qpalkim added Priority: Medium 🔖 우선순위 보통 Type: Refactor ♻️ 리팩토링 labels Feb 15, 2025
@qpalkim qpalkim requested a review from CJewon February 15, 2025 08:39
@qpalkim qpalkim self-assigned this Feb 15, 2025
@vercel
Copy link

vercel bot commented Feb 15, 2025

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

Name Status Preview Comments Updated (UTC)
wine ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 15, 2025 8:39am

Copy link
Collaborator

@CJewon CJewon left a comment

Choose a reason for hiding this comment

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

refactor 과정에서 기능적인 이상 없었습니다 !
또한 코드들도 깔끔하게 정리된것같습니다! 고생하셨습니다 !

@qpalkim qpalkim merged commit 126735b into main Feb 15, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: Medium 🔖 우선순위 보통 Type: Refactor ♻️ 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor : [와인 상세 페이지_컴포넌트] 와인 카드 코드 리팩토링

2 participants