Skip to content
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

hotfix: 강의 시간표상 시간 미정된 강의들에 대한 누락 문제 #80

Merged
merged 4 commits into from
Feb 10, 2024

Conversation

wonslee
Copy link
Contributor

@wonslee wonslee commented Feb 10, 2024

🙋 어떤 PR인가요?

USW 포털 강의 데이터상에서 place_schedule이 "강의실(요일1,2,3)" 패턴이 아닌 "(요일1,2,3)" 패턴인 케이스에서 버그가 발생했었습니다.
기존에는 REGEX 검사를 해서 "강의실(요일1,2,3), ..." 패턴과 일치하는 것만을 시간표 데이터로 응답하였던 것이 문제였습니다.
USW에서 데이터 형태를 바꿀 케이스를 생각해서 만들었던 조건인데, 강의 데이터 적재 로직에서 어느정도 검증된다고 생각해서 제거하게 됐습니다.

📝 작업 상세

  • 강의 데이터 적재 로직 : place_schedule이 존재하지 않는 json 강의 데이터들에 대해 place_schedule을 "null" 스트링으로 저장하는 로직 수정 -> place_schedule 저장 안 하도록
  • 강의 시간표 리스트 조회 로직 : place_schedule 중에서 place가 존재하지 않는 케이스의 경우 "미정"을 반환하도록 수정

🙏 To Reviewers

🧐 체크리스트

  • 본인을 Assign해주시고, 본인을 제외한 백엔드 개발자를 Reviewer로 지정해주세요.
  • 라벨 체크해주세요.
  • .yml 파일 수정 내용이 있다면 공유해주세요!
  • 정상동작하는지, 테스트 통과하는지 다시 한번 확인해주세요.

@wonslee wonslee added the 🚨 hotfix This will not be worked on label Feb 10, 2024
@wonslee wonslee self-assigned this Feb 10, 2024
@wonslee wonslee merged commit 7728096 into develop Feb 10, 2024
1 check passed
@wonslee wonslee deleted the hotfix/lecture_schedule_place_obmission branch February 10, 2024 03:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚨 hotfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant