Skip to content

Datepicker 스타일 변경 #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Feb 7, 2025
Merged

Datepicker 스타일 변경 #1

merged 2 commits into from
Feb 7, 2025

Conversation

jhlee-gsenergy
Copy link
Collaborator

변경의 이유 (Motivation)

  • Datepicker Day 일 제거
  • 현재 month에 포함되지 않는 Day 스타일 변경
  • 레포지토리 관리를 위한 .gitignore 수정

변경의 종류 (Type of Change)

  • 버그 수정 (Bug fix)
  • 새로운 기능 (New feature)
  • 기능 변경 (Feature change)
  • 리팩토링 (Refactoring)
  • 브레이킹 체인지 (Breaking change)
  • 문서 업데이트 (Documentation update)

변경의 내용 (Change List)

  • Datepicker Day 일 제거
  • 현재 month에 포함되지 않는 Day 스타일 변경
  • 레포지토리 관리를 위한 .gitignore 수정

관련 이슈 (Related Issues)

기대 효과(Intended Effects)

  • Datepicker 디자인을 원하는대로 변경이 가능합니다.

코드 품질 (Code Quality)

  • 변경사항을 자체적으로 리뷰하였습니다. (I have self-reviewed my code)
  • 변경사항을 로컬에서 테스트 했습니다. (I have tested my changes locally)
  • 코드의 이해를 돕기 위한 주석을 추가하였습니다. (I have added comments to help understand the code)
  • 적절한 단위 테스트를 추가하였습니다. (I have added appropriate unit tests)
  • 커밋은 적절한 크기로 나누어졌습니다. (My commits are atomic and appropriately sized)

테스트 방법(How Has This Been Tested?)

  • frontend에서 dependency 주입 후 yarn start

리뷰어 체크리스트 (Reviewer Checklist)

  • 이 PR 의 변경 사항을 로컬에서 테스트했습니다. (I have tested the changes in this PR locally)
  • 변경 사항이 코딩 스타일 가이드를 준수하고 있습니다. (The changes in this PR comply with the coding style guide)
  • 보안 가이드라인을 준수하고 있습니다. (The changes in this PR comply with the security guidelines)
  • 전체 단위테스트가 통과되었습니다. (All unit tests pass)

@jhlee-gsenergy jhlee-gsenergy self-assigned this Feb 7, 2025
Copy link

@HOWISYUNI HOWISYUNI left a comment

Choose a reason for hiding this comment

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

확인했습니다 감사합니다 😄

@jhlee-gsenergy jhlee-gsenergy merged commit 20fd4e2 into main Feb 7, 2025
@jhlee-gsenergy jhlee-gsenergy deleted the dev branch February 7, 2025 01:56
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.

2 participants