-
Notifications
You must be signed in to change notification settings - Fork 0
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
[DEV-28] 커스텀 수강과목 수정 api 분리 생성 삭제 #249
The head ref may contain hidden characters: "refactor/DEV-28-\uCEE4\uC2A4\uD140-\uC218\uAC15\uACFC\uBAA9-\uC218\uC815-API-\uBD84\uB9AC-\uC0DD\uC131-\uC0AD\uC81C"
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.
확인했습니다! 분리된 UseCase가 기존 UseCase와 명확히 분리되도록 정의만 되면 될거 같습니다!
public void generateTakenLecture(@LoginUser Long userId, | ||
@Valid @RequestBody GenerateTakenLectureRequest generateTakenLectureRequest) { | ||
generateTakenLectureUseCase.generateTakenLecture(userId, generateTakenLectureRequest.getLectureId()); |
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.
컨트롤러 메서드 명을 더 명확하게 해주는 것이 좋을거 같습니다!
ex. addCustomizedTakenLecture
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.
} | ||
|
||
@DeleteMapping("{taken-lecture-id}") |
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.
@DeleteMapping("{taken-lecture-id}") | |
@DeleteMapping("{takenLectureId}") |
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.
@UseCase | ||
@Transactional | ||
@RequiredArgsConstructor | ||
public class DeleteTakenLectureServiceById implements DeleteTakenLectureUseCase { |
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.
public class DeleteTakenLectureServiceById implements DeleteTakenLectureUseCase { | |
public class DeleteTakenLectureService implements DeleteTakenLectureUseCase { |
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.
package com.plzgraduate.myongjigraduatebe.takenlecture.application.usecase.save; | ||
|
||
|
||
public interface GenerateTakenLectureUseCase { |
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.
pdf 파싱으로 생성되는 TakenLecture 유스케이스와 구분할 수 있도록 행동을 명확하게 정의한 유스케이스명을 사용하는 것이 좋을거 같습니다!
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.
@@ -37,6 +39,7 @@ public class TakenLectureJpaEntity extends TimeBaseEntity { | |||
|
|||
private Integer year; | |||
|
|||
@Enumerated(EnumType.STRING) |
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.
해당 변경으로 인한 실제 운영디비의 값 변경에 대한 내용은 따로 문서화 부탁드립니다!
Quality Gate passedIssues Measures |
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.
👍
Issue
✅ 작업 내용
변경된 api 명세