Skip to content

Conversation

@hyeonjiroh
Copy link
Collaborator

#️⃣ Issue Number

#129



📝 요약(Summary)

DonationsList

  • 플레이스 홀더 부분에 사용된 매직 넘버를 의미 있는 이름의 상수로 대체

DonationModalContent

  • 조건을 상수에 할당해서 할당된 boolean값을 setter 함수에 넘겨주는 방식으로 변경
  • input 태그를 포함하는 부모 태그를 div 태그에서 form 태그로 변경


🛠️ PR 유형

어떤 변경 사항이 있나요?

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


@hyeonjiroh hyeonjiroh self-assigned this Feb 5, 2025
@vercel
Copy link

vercel bot commented Feb 5, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fandom-k ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 5, 2025 2:34pm

@github-actions github-actions bot requested review from jihye5081 and juha399 February 5, 2025 10:36
Copy link
Collaborator

@jihye5081 jihye5081 left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다~!!

@juha399 juha399 merged commit 3f99766 into main Feb 5, 2025
2 checks passed
@juha399 juha399 deleted the refactor/#129/Refactor-Donation-Part branch February 5, 2025 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants