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

refactor: 졸업 학점 계산 Catalog 테이블 학과 공통 처리 및 major 우선 로직 반영 #1208

Conversation

seongjae6751
Copy link
Contributor

🔥 연관 이슈

🚀 작업 내용

1.catalog 테이블을 이용하는 코드 중 catalog 테이블의 major_id를 먼저 확인 후 null이면 department_id를 가져오게 했습니다.
2. year로 같이 조회하게 다시 원상복구 했습니다.

💬 리뷰 중점사항

@seongjae6751 seongjae6751 self-assigned this Jan 30, 2025
@github-actions github-actions bot added the 기능 새로운 기능을 개발합니다. label Jan 30, 2025
@seongjae6751 seongjae6751 changed the title Feature/1204 catalog department major handling refactor: 졸업 학점 계산 Catalog 테이블 학과 공통 처리 및 major 우선 로직 반영 Jan 30, 2025
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

@Soundbar91 Soundbar91 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 ~
아마 브랜치에 year 반영되서, 머지되고 테스트 하면 될 거 같아요.

@seongjae6751 seongjae6751 merged commit c919e83 into feature/1140-graduation-credit-calculator Feb 7, 2025
3 checks passed
@seongjae6751 seongjae6751 deleted the feature/1204-catalog-department-major-handling branch February 7, 2025 11:54
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