Skip to content

[#90] Refactor/와인목록페이지 와인목록#94

Merged
LMS10 merged 11 commits intomainfrom
refactor/와인목록페이지_와인목록
Feb 16, 2025

Hidden character warning

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

[#90] Refactor/와인목록페이지 와인목록#94
LMS10 merged 11 commits intomainfrom
refactor/와인목록페이지_와인목록

Conversation

@LMS10
Copy link
Owner

@LMS10 LMS10 commented Feb 16, 2025

#️⃣ Issue

📝 Description

  • 가격 필터 최종값만 요청 보내도록 이벤트 핸들러 추가
  • 필터 css 개선
  • 와인 등록 모달 필드 다 안 채웠을 때 alert -> 버튼 disabled로 변경
  • 와인 등록 성공/실패 메시지 토스트 메세지
  • 추천 와인 목록 fetch 실패 시 Refresh 컴포넌트 추가하여 에러 처리 개선
  • 와인 목록 fetch 실패 시 Refresh 컴포넌트 추가하여 에러 처리 개선
  • 와인 목록 로딩 시 스켈레톤 UI 적용하여 로딩 처리 개선
  • 전역 메타데이터 설정
토스트 에러처리 로딩처리

🌀 PR Type

어떤 변경 사항이 있나요?

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

✅ Checklist

PR Checklist

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

  • Branch Convention 확인

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

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

Test Checklist

  • 로컬 작동 확인

Additional Notes

  • (없음)

@LMS10 LMS10 added Priority: Medium 🔖 우선순위 보통 Type: Refactor ♻️ 리팩토링 labels Feb 16, 2025
@LMS10 LMS10 requested a review from qpalkim February 16, 2025 09:24
@LMS10 LMS10 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 9:49am

@qpalkim
Copy link
Collaborator

qpalkim commented Feb 16, 2025

로컬 동작 확인했습니다! 수고하셨습니다~!👍

@LMS10 LMS10 merged commit 9c060f0 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