-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: 시간표 조회 API 이수 구분 반환 추가 #1150
Merged
Soundbar91
merged 6 commits into
feature/1140-graduation-credit-calculator
from
feat/response-add-coures-type
Dec 29, 2024
Merged
feat: 시간표 조회 API 이수 구분 반환 추가 #1150
Soundbar91
merged 6 commits into
feature/1140-graduation-credit-calculator
from
feat/response-add-coures-type
Dec 29, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Soundbar91
requested review from
seongjae6751,
daheeParkk,
kwoo28 and
duehee
December 21, 2024 16:23
duehee
approved these changes
Dec 23, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
전체적으로 잘 작성해주신 거 같아요!
작은 수정사항 하나만 확인해주세요 :D
public CourseType HRD_필수() { | ||
return courseTypeRepository.save( | ||
CourseType.builder() | ||
.name("MSC 필수") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C
Suggested change
.name("MSC 필수") | |
.name("HRD 필수") |
👀
Soundbar91
merged commit Dec 29, 2024
3aada93
into
feature/1140-graduation-credit-calculator
2 checks passed
duehee
pushed a commit
that referenced
this pull request
Dec 30, 2024
* feat: repository 추가 * feat: exception 추가 * feat: 이수 구분 반환값 추가 * feat: coursetype 반환값 추가 * test: 테스트 코드 수정 * chore: 리뷰 반영
duehee
added a commit
that referenced
this pull request
Dec 30, 2024
* feat : 엑셀 파일 최신화 * feat : 리팩토링 진행 및 패키지 이동 * chore : 안 쓰는 import 제거 및 정리 * chore : 관규님 리뷰 반영 * chore : 개행 추가 * feat: lecture 컬럼 크기 수정 flyway 추가 (#1142) * fix: 식단 이미지 알림 롤백 (#1145) * fix: 스케줄러 cron식 수정 * fix: 식단 이미지 업로드 알림 롤백 * fix : 동계방학 생협 운영시간 업데이트 (#1144) * fix : 생협 동계학기 운영 시간 수정 * fix : 생협 동계학기 운영 시간 수정 * chore : 공백 추가 * fix: 세탁소 전화번호 수정 * feat: 스웨거 그룹화 (#1138) * chore: swagger 패키지 생성 및 클래스 이동 * feat: 비즈니스팀 API 그룹화 * chore: 비즈니스 API 추가 * feat: 캠퍼스팀 API 그룹화 * feat: 유저팀 API 그룹화 * feat: ABTEST API 그룹화 * feat: BCSD API 그룹화 * chore: 비즈니스팀 API 추가 * feat: 어드민 API 그룹화 * chore: 유저팀 API 추가 * feat: 로그인 API 그룹화 및 그룹 이름 변경 * refactor: 패키지 경로 enum화 * refactor: 중복 코드 메소드화 * refactor: 그룹 스웨거 파일 분할 * chore: 미사용 import 삭제 * chore: 메소드 명 변경 * chore: 리뷰 반영 * fix: 안경점, 우편취급국 운영시간 추가, 대즐 운영시간 오타 수정 (#1151) * feat: flyway추가 & model 관련 추가 및 수정 * feat: 시간표 조회 API 이수 구분 반환 추가 (#1150) * feat: repository 추가 * feat: exception 추가 * feat: 이수 구분 반환값 추가 * feat: coursetype 반환값 추가 * test: 테스트 코드 수정 * chore: 리뷰 반영 * fix: department 중복 수정 * fix: department 조회 로직 변경 * rebase : 충돌 해결 * chore : 현수님 리뷰 반영 --------- Co-authored-by: 신관규 <[email protected]> Co-authored-by: 허준기 <[email protected]> Co-authored-by: Hwang HyeonSik <[email protected]> Co-authored-by: 김원경 <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔥 연관 이슈
🚀 작업 내용
💬 리뷰 중점사항
student 테이블의 year, department와 timetable_lecture의 code를 이용해서 catalog 테이블의 id를 얻습니다.
catalog의 id를 이용해서 course_type_id를 찾아서 timetable_lecture에 저장하도록 작성했습니다.
timetable_lecture의 course_type_id를 이용해서 이수 구분을 응답으로 반환하도록 작성했습니다.
노후화 이전 PR