Skip to content

[#65] Feat/와인상세페이지 수정/삭제#69

Merged
qpalkim merged 8 commits intomainfrom
feat/와인상세페이지_수정/삭제
Feb 14, 2025

Hidden character warning

The head ref may contain hidden characters: "feat/\uc640\uc778\uc0c1\uc138\ud398\uc774\uc9c0_\uc218\uc815/\uc0ad\uc81c"
Merged

[#65] Feat/와인상세페이지 수정/삭제#69
qpalkim merged 8 commits intomainfrom
feat/와인상세페이지_수정/삭제

Conversation

@qpalkim
Copy link
Collaborator

@qpalkim qpalkim commented Feb 13, 2025

#️⃣ Issue

📝 Description

내가 작성한 리뷰의 케밥 아이콘을 클릭 시 "수정하기", "삭제하기" 드롭다운이 나타나며 해당하는 모달을 연결함

  • "수정하기"를 클릭하면 수정하기 모달이 나타나고, 리뷰를 수정할 수 있음
  • "삭제하기"를 클릭하면 삭제 확인 모달이 나타나고, "삭제하기"를 클릭했을 때 리뷰가 정상적으로 삭제됨
  • ReviewDropdown 컴포넌에 데이터 페치 작업을 불러와 PATCH, DELETE 요청을 연결하였음

🌀 PR Type

어떤 변경 사항이 있나요?

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

✅ Checklist

PR Checklist

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

  • Branch Convention 확인

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

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

Test Checklist

  • 로컬 작동 확인

Additional Notes

  • 리뷰 수정 완료 시, 리뷰 수정 시간이 나타나도록 수정함
  • fetchWineDetail.ts 파일을 만들어 /wines/[id] API의 데이터 페치를 따로 분리하였음(로딩, 에러 처리 개선, 코드 간결화 등 추후 보완할 예정)
  • 평점 실시간 반영

@qpalkim qpalkim added Priority: Medium 🔖 우선순위 보통 Type: Feature ✨ 신규 기능 labels Feb 13, 2025
@qpalkim qpalkim requested a review from LMS10 February 13, 2025 10:28
@qpalkim qpalkim self-assigned this Feb 13, 2025
@vercel
Copy link

vercel bot commented Feb 13, 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 14, 2025 11:21am

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 a55a46f into main Feb 14, 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: Feature ✨ 신규 기능

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feat : [와인 상세 페이지_수정/삭제] 수정하기 및 삭제하기 모달 작업

2 participants