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

fix: 메인시간표 필수로 수정 #890

Merged
merged 1 commit into from
Sep 15, 2024
Merged

Conversation

kwoo28
Copy link
Contributor

@kwoo28 kwoo28 commented Sep 14, 2024

🔥 연관 이슈

🚀 작업 내용

1. 로직 수정
학기에 메인시간표 1개를 필수로 두게했지만, 현재 발생한 문제는 기존 시간표 프레임 수정할때 메인시간표를 false로 변경하고 나면 메인시간표를 1개를 필수로 두어야하는 정합성에 불일치가 생겨 로직 수정했습니다.

2. DB데이터 수정
image

현재 발생한 문제로 인해 stage DB에 {user_id=1020, semester_id=10}과 {user_id=1782, semester_id=11} 두개의 데이터에서 메인시간표가 존재하지 않는 정합성 불일치가 존재합니다.

그래서 {user_id=1020, semester_id=10}에 해당하는 id=1046데이터를 is_main=1로 변경하고 {user_id=1782, semester_id=11}에 해당하는 id=2048을 is_main=1로 변경하고자합니다.

💬 리뷰 중점사항

@kwoo28 kwoo28 self-assigned this Sep 14, 2024
@github-actions github-actions bot added the 버그 정상적으로 동작하지 않는 문제상황입니다. label Sep 14, 2024
Copy link

Unit Test Results

286 tests   285 ✔️  1m 5s ⏱️
  31 suites      1 💤
  31 files        0

Results for commit a1397af.

Copy link
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.

예외처리 굿😄

Copy link
Contributor

@seongjae6751 seongjae6751 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@kwoo28 kwoo28 merged commit c354410 into develop Sep 15, 2024
6 checks passed
@kwoo28 kwoo28 deleted the fix/879-put-timetableFrame branch September 15, 2024 06:26
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.

메인시간표 필수 사항으로 변경
3 participants