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

feat: 시간표 조회 API 이수 구분 반환 추가 #1150

Conversation

Soundbar91
Copy link
Contributor

🔥 연관 이슈

🚀 작업 내용

  1. GET /v2/timetables/lecture의 응답에 이수 구분 추가
  2. 이수 구분 관련 Repository, Exception 작업
  3. POST /v2/timetables/lecture의 요청에서 year, department, code를 이용해서 이수 구분 id 저장
  4. 테스트 코드 수정

💬 리뷰 중점사항

student 테이블의 year, department와 timetable_lecture의 code를 이용해서 catalog 테이블의 id를 얻습니다.
catalog의 id를 이용해서 course_type_id를 찾아서 timetable_lecture에 저장하도록 작성했습니다.

timetable_lecture의 course_type_id를 이용해서 이수 구분을 응답으로 반환하도록 작성했습니다.
노후화 이전 PR

@Soundbar91 Soundbar91 self-assigned this Dec 21, 2024
@Soundbar91 Soundbar91 added 기능 새로운 기능을 개발합니다. Team User 유저 팀에서 작업할 이슈입니다 labels Dec 21, 2024
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.

전체적으로 잘 작성해주신 거 같아요!
작은 수정사항 하나만 확인해주세요 :D

public CourseType HRD_필수() {
return courseTypeRepository.save(
CourseType.builder()
.name("MSC 필수")
Copy link
Contributor

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 Soundbar91 merged commit 3aada93 into feature/1140-graduation-credit-calculator Dec 29, 2024
2 checks passed
@Soundbar91 Soundbar91 deleted the feat/response-add-coures-type branch December 29, 2024 16:11
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
Labels
Team User 유저 팀에서 작업할 이슈입니다 기능 새로운 기능을 개발합니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants