-
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 이수 구분 반환 추가 #932
feat: 시간표 조회 API 이수 구분 반환 추가 #932
Conversation
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.
음... 전 고칠점이 없는 것 같습니다..! 깔끔하게 잘 작성하신것같아요. 수고 많으셨습니다!
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.
우와 첫 PR~ 고생하셨습니다!!
코멘트 하나 있으니 확인해주세요!
); | ||
} | ||
|
||
public CourseType HRD_필수() { |
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
HRD 필수인데 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.
실수로 아래를 안 땡겨서 리뷰를.. 아래 save쪽이에요
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.
LectureFixture에 HRD_개론
이름으로 되어 있고 내용은 컴퓨팅사고
이길레 기존거는 냅두고 CourseType를 다음과 같이 작성했습니다..!
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.
아래 github action flyway에서 테스트 오류 난 거 같은데 한 번만 확인해주세요~
CourseType courseType = timetableLectureRequest.courseTypeId() == null ? | ||
null : courseTypeRepository.getCourseTypeById(timetableLectureRequest.courseTypeId()); | ||
|
||
CourseType courseType = lecture == null ? |
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
라인 포맷팅 안 되어있나요?? 2줄로 나뉠 로직이 아닌 거 같아서요👀
String code = lecture.getCode(); | ||
|
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
공백 한 줄 지워주세영
…-graduation-credit-calculator
fbefe93
into
feature/929-graduation-credit-calculator
🔥 연관 이슈
🚀 작업 내용
GET /v2/timetables/lecture
의 응답에이수 구분
추가POST /v2/timetables/lecture
의 요청에서 year, department, code를 이용해서이수 구분 id
저장💬 리뷰 중점사항
student 테이블의 year, department와 timetable_lecture의 code를 이용해서 catalog 테이블의 id를 얻습니다.
catalog의 id를 이용해서 course_type_id를 찾아서 timetable_lecture에 저장하도록 작성했습니다.
timetable_lecture의 course_type_id를 이용해서
이수 구분
을 응답으로 반환하도록 작성했습니다.첫 PR 리뷰 부탁드립니다..!