Skip to content

[#98] Refactor/공통컴포넌트 모달#101

Merged
park521 merged 5 commits intomainfrom
refactor/공통컴포넌트_모달
Feb 18, 2025

Hidden character warning

The head ref may contain hidden characters: "refactor/\uacf5\ud1b5\ucef4\ud3ec\ub10c\ud2b8_\ubaa8\ub2ec"
Merged

[#98] Refactor/공통컴포넌트 모달#101
park521 merged 5 commits intomainfrom
refactor/공통컴포넌트_모달

Conversation

@park521
Copy link
Collaborator

@park521 park521 commented Feb 18, 2025

#️⃣ Issue

📝 Description

모달 컴포넌트 버그 해결 및 리팩토링 작업에 관한 PR입니다.

  • 모달 컴포넌트 초기값 전달
  • 리뷰수정/와인수정 모달에서 변경 값을 저장하지 않고 모달을 닫았을 때 초기화 되지 않는 버그 해결
  • 와인수정 모달에서 필드 하나라도 비어있으면 버튼 비활성화

🌀 PR Type

어떤 변경 사항이 있나요?

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

✅ Checklist

PR Checklist

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

  • Branch Convention 확인

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

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

Test Checklist

  • 로컬 작동 확인

Additional Notes

  • (없음)

@park521 park521 requested a review from LMS10 February 18, 2025 02:37
@vercel
Copy link

vercel bot commented Feb 18, 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 18, 2025 4:55am

@park521 park521 changed the title 🎨 refactor : 모달 리팩터 및 버그 개선 [#98] Refactor/공통컴포넌트 모달 Feb 18, 2025
@park521 park521 self-assigned this Feb 18, 2025
@park521 park521 added Priority: Low 🐢 우선순위 낮음 Type: Bug 🐛 버그 Type: Refactor ♻️ 리팩토링 labels Feb 18, 2025
@park521 park521 linked an issue Feb 18, 2025 that may be closed by this pull request
2 tasks
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.

수고하셨습니다 ~

@park521 park521 merged commit 26de870 into main Feb 18, 2025
3 checks passed
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