Skip to content

[#88] Refactor/와인상세페이지 리뷰 #92

Merged
qpalkim merged 10 commits intomainfrom
refactor/와인상세페이지_리뷰
Feb 16, 2025

Hidden character warning

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

[#88] Refactor/와인상세페이지 리뷰 #92
qpalkim merged 10 commits intomainfrom
refactor/와인상세페이지_리뷰

Conversation

@qpalkim
Copy link
Collaborator

@qpalkim qpalkim commented Feb 16, 2025

#️⃣ Issue

📝 Description

  • 리뷰 등록, 수정, 삭제 시 success 토스트 구현
  • 리뷰 등록, 수정, 삭제에 실패했을 시 error 토스트 구현
  • 리뷰 모달이 아니면 컨트롤바 커서 default로 수정함
  • 리뷰 등록 모달 등록하거나 창 닫을 시 다시 리뷰 등록 모달 열었을 때 기존에 입력했던 값 초기화
  • 리뷰 수정, 삭제 시 새로고침 없이 4가지 맛 평균, 많이 선택된 3가지 향, 별점 컴포넌트 바로 반영되도록
  • 좋아요 누르거나 취소 시 토스트 구현
  • 리뷰 스켈레톤 UI
  • 전반적인 코드 리팩토링
  • 메타 태그 추후 해결할 예정....

🌀 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 16, 2025
@qpalkim qpalkim requested a review from LMS10 February 16, 2025 08:10
@qpalkim qpalkim self-assigned this Feb 16, 2025
@vercel
Copy link

vercel bot commented Feb 16, 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 16, 2025 8:54am

Copy link
Owner

@LMS10 LMS10 left a comment

Choose a reason for hiding this comment

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

테스트 완료했습니다~! 수고하셨습니다👍🏻

@qpalkim qpalkim merged commit f197ebc into main Feb 16, 2025
3 checks passed
@LMS10 LMS10 added the Type: Bug 🐛 버그 label Feb 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

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

2 participants