Skip to content

Conversation

@huiwoo-jo
Copy link
Contributor

@huiwoo-jo huiwoo-jo commented Jan 16, 2025

📮 관련 이슈

✍️ 구현 내용

  • 기존 하드 코딩된 학사 일정의 범위를 조정하였습니다.
  • currentYear을 지정하고 LocalDate와 비교하여 다를 경우 출력 범위를 조정합니다.
  • 학사 일정에 변동이 있는 경우가 있어 로컬 저장 방식을 제거하였습니다.
    이는 추후 로컬 저장 데이터와 현재 데이터와 비교하여 다를 경우만 로컬 데이터를 변경하여 로드 하는 방식이 효율적일듯 합니다.
  • currentYear은 월 단위만 유효하기 떄문에 2월의 윤달은 체크하지 않았습니다.
  • firstLaunchCheck로 currentYear을 초기 설정하는 방식도 있으나 LocalDate와의 비교 부분이 있어 nullCheck를 비교와 같이 넣어도 충분할거 같습니다.

📷 구현 영상

KakaoTalk_Video_2025-01-16-16-08-22.mp4

✔️ 확인 사항

  • 컨벤션에 맞는 PR 타이틀
  • 관련 이슈 연결
  • PR 관련 정보 연결 (작업자, 라벨, 마일스톤 등)
  • Github Action 통과

@huiwoo-jo huiwoo-jo requested a review from m6z1 January 16, 2025 07:09
@huiwoo-jo huiwoo-jo self-assigned this Jan 16, 2025
@m6z1 m6z1 merged commit ce50fee into develop Feb 6, 2025
1 check passed
@m6z1 m6z1 deleted the chore/schedule-date-range branch February 6, 2025 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

학사 일정 조정

3 participants