Skip to content

[#99] Refactor/와인목록페이지#100

Merged
LMS10 merged 6 commits intomainfrom
refactor/와인목록페이지
Feb 17, 2025

Hidden character warning

The head ref may contain hidden characters: "refactor/\uc640\uc778\ubaa9\ub85d\ud398\uc774\uc9c0"
Merged

[#99] Refactor/와인목록페이지#100
LMS10 merged 6 commits intomainfrom
refactor/와인목록페이지

Conversation

@LMS10
Copy link
Owner

@LMS10 LMS10 commented Feb 17, 2025

#️⃣ Issue

📝 Description

  • 와인 목록 초기 로딩은 스켈레톤, 추가 로딩은 스피너로 로딩 처리 개선
  • Esc 또는 모달 외부 클릭해서 모달 닫았을 때 리뷰 남기기, 와인 등록하기 모달 필드 초기화 안 되는 버그 해결
  • 와인 등록/수정 모달 가격 필드 200만원 제한 및 천 단위로 변환
  • 버튼 호버 시 배경색 전환 효과 추가

🌀 PR Type

어떤 변경 사항이 있나요?

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

✅ Checklist

PR Checklist

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

  • Branch Convention 확인

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

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

Test Checklist

  • 로컬 작동 확인

Additional Notes

  • (없음)

@LMS10 LMS10 requested a review from eunkyung01 February 17, 2025 06:49
@LMS10 LMS10 self-assigned this Feb 17, 2025
@vercel
Copy link

vercel bot commented Feb 17, 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 17, 2025 6:49am

@LMS10 LMS10 requested review from park521 and removed request for eunkyung01 February 17, 2025 07:05
@LMS10 LMS10 merged commit 4f3e9fc into main Feb 17, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor : [와인목록페이지] 버그 해결 및 리팩토링 작업

2 participants