Skip to content

hotfix: Timetable NPE 수정 (main)#1570

Merged
Soundbar91 merged 2 commits into
mainfrom
hotfix/timetable-npe
May 20, 2025
Merged

hotfix: Timetable NPE 수정 (main)#1570
Soundbar91 merged 2 commits into
mainfrom
hotfix/timetable-npe

Conversation

@Soundbar91
Copy link
Copy Markdown
Collaborator

🔥 연관 이슈

🚀 작업 내용

  • timetable에서 발생하는 NPE를 수정했습니다.
    • 졸업학점 계산기 엑셀 업로드 과정에서 정규강의가 아닌 강의는 임의로 생성되서 timetable_lecture에 저장
    • 이 과정에서 온라인 강의 같은 수업의 classTime이 Null로 들어갔음
    • 온라인 강의의 classTime은 []으로 들어가야 하므로, 생성 로직 수정
    • 응답값 반환과정에서도 빈 리스트 처리

해당 PR은 Main 브랜치를 향하고 있습니다.

@Soundbar91 Soundbar91 self-assigned this May 20, 2025
@Soundbar91 Soundbar91 added the 버그 정상적으로 동작하지 않는 문제상황입니다. label May 20, 2025
@Soundbar91 Soundbar91 changed the title hofix: Timetable NPE 수정 (main) hoTfix: Timetable NPE 수정 (main) May 20, 2025
@Soundbar91 Soundbar91 changed the title hoTfix: Timetable NPE 수정 (main) hotfix: Timetable NPE 수정 (main) May 20, 2025
Copy link
Copy Markdown
Collaborator

@ImTotem ImTotem left a comment

Choose a reason for hiding this comment

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

🌊🏊💦어푸어푸🌊🏊💦

Copy link
Copy Markdown
Contributor

@duehee duehee left a comment

Choose a reason for hiding this comment

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

재앙
졸학계야미안해.. 분명 생성되는 수업 문제 없이 처리했던 거 같은데, 오류가 생겼네요.... ..

@github-actions
Copy link
Copy Markdown

Unit Test Results

359 tests   358 ✔️  1m 44s ⏱️
  49 suites      1 💤
  49 files        0

Results for commit 2b0eea2.

@Soundbar91 Soundbar91 merged commit afe3673 into main May 20, 2025
10 of 11 checks passed
@Soundbar91 Soundbar91 deleted the hotfix/timetable-npe branch May 20, 2025 05:13
@Soundbar91 Soundbar91 restored the hotfix/timetable-npe branch May 20, 2025 05:16
@Soundbar91 Soundbar91 deleted the hotfix/timetable-npe branch May 20, 2025 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

버그 정상적으로 동작하지 않는 문제상황입니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants