Skip to content

[FIX] 캘린더 모달 오늘 버튼 동작 정상화 #436

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
Apr 4, 2025

Conversation

Kjiw0n
Copy link
Contributor

@Kjiw0n Kjiw0n commented Apr 2, 2025

작업 내용 🧑‍💻

  • DateCorrectionModal 오늘 버튼 동작 정상화

알게된 점 🚀

기록하며 개발하기!

  • fullcalender의 ReactDatePickerCustomHeaderProps에 change Month, Year 함수 통해 현재 보고있는 달력 뷰 조정 가능

리뷰 요구사항 💬

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

  • fullcalendar에서 오늘 기능은 없어도 뷰 조정 함수를 지원해줘서 저번 의논보다 굉장히 간단하게 끝났습니다~

관련 이슈

close #428

스크린샷 (선택)

Copy link

github-actions bot commented Apr 2, 2025

Copy link
Member

@seong-hui seong-hui left a comment

Choose a reason for hiding this comment

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

DateCorrectionModal의 오늘 버튼 동작 잘 되네요!! 고생 많으셨습니다. 다만 캘린더의 날짜가 이동해도 캘린더 모달의 날짜는 계속 오늘 날짜라서 해당 코드를 살펴보니 FullCalendarBox에서 DateCorrectionModaldatenew Date().toISOString()로 전달하고 있어서 그런 것으로 확인하였습니다. 따라서 해당 부분을 startDate로 수정하면 캘린더 뷰에서 보이는 시작 날짜가 캘린더 모달을 열었을때도 올바르게 보일 것 같습니다!!

  • 문제 상황 : 현재는 다른 날짜로 이동해도 모달의 날짜는 항상 오늘을 보여주고 있음.
image
  • 수정할 코드 FullCalendarBoxDateCorrectionModal
<DateCorrectionModal
			date={startDate}
			onClick={handleCalendarPopup}
			top={9.8}
			right={0.8}
			handleCurrentDate={handleChangeDate}
		/>

@github-project-automation github-project-automation bot moved this from Todo to In Progress in @DAWM-FE Apr 3, 2025
@Kjiw0n
Copy link
Contributor Author

Kjiw0n commented Apr 4, 2025

연동 깜빡한 거 체크해주셔서 감사합니다 👍 왕ㄸㅏ봉

Copy link

github-actions bot commented Apr 4, 2025

@Kjiw0n Kjiw0n merged commit 7ab0cee into develop Apr 4, 2025
4 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in @DAWM-FE Apr 4, 2025
@Kjiw0n Kjiw0n deleted the fix/#428/calendar-modal-qa branch April 4, 2025 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FIX] 캘린더모달 qa 반영
2 participants