Skip to content
New issue

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

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

Already on GitHub? # to your account

[#118] 날짜 오류 수정 #119

Merged
merged 1 commit into from
Jun 3, 2024
Merged

[#118] 날짜 오류 수정 #119

merged 1 commit into from
Jun 3, 2024

Conversation

KkomSang
Copy link
Collaborator

@KkomSang KkomSang commented Jun 3, 2024

#️⃣ 연관된 이슈

Resolves #118

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요 (이미지 첨부 가능)

31일이 없는 달에 31일까지의 데이터를 조회하기에 발생하는 오류를 해결했습니다.
달의 마지막 날을 31이 아닌, currentDate.lengthOfMonth()로 수정했습니다!

스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

✅ Check List

  • PR 제목을 커밋 규칙에 맞게 작성했는가?
  • PR에 해당되는 Issue를 연결했는가?
  • 적절한 라벨을 설정했는가?
  • 작업한 사람을 모두 Assign했는가?

@KkomSang KkomSang added the fix 기능 및 버그를 수정합니다 label Jun 3, 2024
@KkomSang KkomSang self-assigned this Jun 3, 2024
@KkomSang KkomSang requested a review from yel-m June 3, 2024 18:26
@yel-m
Copy link
Member

yel-m commented Jun 3, 2024

화긴했습니다!! 새벽까지 수고 많으셨습니당!!🥹

@yel-m yel-m merged commit e176b1a into develop Jun 3, 2024
1 check passed
@yel-m yel-m deleted the fix/118 branch June 3, 2024 23:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
fix 기능 및 버그를 수정합니다
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants