-
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] 지원 서류 삭제 기능 #177
Merged
Merged
[feat] 지원 서류 삭제 기능 #177
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
hyunihs
requested review from
suhhyun524,
mushroom1324,
haen-su and
mirageoasis
as code owners
July 24, 2024 09:09
hyunihs
requested review from
itsme-shawn,
YoungGyo-00,
yoonsseo and
letskuku
and removed request for
mirageoasis,
haen-su and
suhhyun524
July 24, 2024 09:10
mirageoasis
reviewed
Jul 24, 2024
src/main/java/ceos/backend/domain/recruitment/service/RecruitmentService.java
Show resolved
Hide resolved
letskuku
approved these changes
Jul 25, 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.
고생하셨습니다~ 프로젝트 구조가 전반적으로 제가 해오던 방식과는 조금 달라서 재밌게 봤습니다👍
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.
커스텀 예외가 싱글톤 패턴을 이용한 단일 인스턴스 생성으로 구현되어있군요
좋은 방법이네요👍
hyunihs
added a commit
that referenced
this pull request
Aug 14, 2024
* [feat] 지원 서류 삭제 기능 (#177) * [feat] 지원 서류 삭제 * [chore] update CODEOWNERS * [feat] 지원서 목록 보기 API required false인 parameter들에 대해 default value 설정 * [test] 지원서 목록 보기 API required false인 parameter들에 대한 테스트 코드 작성 * [feat] 지원서 목록 보기 API enum 타입인 parameter들에 대해 allowable values 설정 * [test] 지원서 목록 보기 API enum 타입인 parameter들에 대해 테스트 코드 작성 * [fix] 지원서 엑셀 파일 생성 시각 API 테스트 status code 수정 * [revert] 코드 수정 내역 되돌림 * [feat] 지원서 목록 보기 API required false인 parameter들에 대해 default value 설정 * [test] 지원서 목록 보기 API required false인 parameter들에 대한 테스트 코드 작성 * [feat] 지원서 목록 보기 API enum 타입인 parameter들에 대해 allowable values 설정 * [test] 지원서 목록 보기 API enum 타입인 parameter들에 대해 테스트 코드 작성 * [fix] 지원서 엑셀 파일 생성 시각 API 테스트 status code 수정 * [revert] 코드 수정 내역 되돌림 * [feat] 지원서 목록 보기 API required false인 parameter들에 대해 default value 설정 * [test] 지원서 목록 보기 API required false인 parameter들에 대한 테스트 코드 작성 * [feat] 지원서 목록 보기 API enum 타입인 parameter들에 대해 allowable values 설정 * [test] 지원서 목록 보기 API enum 타입인 parameter들에 대해 테스트 코드 작성 * [fix] 지원서 엑셀 파일 생성 시각 API 테스트 status code 수정 * [feat] 면접 참여 여부 확인용 DTO 생성 #180 * [feat] 최종 활동 여부 확인용 DTO 생성 #180 * [feat] 불가능 사유 unableReason 필드 및 해당 필드 업데이트 로직 추가 #180 * [feat] 최종 활동 여부 확인용 DTO 생성 #180 * [feat] 면접 참여 가능 여부 확인과 활동 가능 여부 확인 Service 로직 작성 #180 * [feat] 활동 가능 여부 확인 시 최종 합격 여부 검증하는 validator 작성 #180 * feat: 면접 참여 및 활동 가능 여부 확인용 Controller 작성 * feat: 면접 참여 및 활동 가능 여부 확인용 Controller 작성 #180 * [refact] 회고록 목록 보기 response dto 필드 수정 * [refact] 회고록 상세 보기/수정 response dto 필드 수정 * [refact] 리프레시 토큰 발급 로직 수정 (#186) * [refact] 리프레시토큰 발급 로직 수정 #185 * [refact] 상수 추가 --------- Co-authored-by: yoonsseo <[email protected]> Co-authored-by: letskuku <[email protected]> Co-authored-by: Yujeong Lee <[email protected]> Co-authored-by: 이윤서 <[email protected]> Co-authored-by: suhhyun <[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.
💡 [feat] 지원 서류 삭제 기능
📝 Description
🌲 Working Branch
feat/deleteApplication
📖 Related Issues
이슈 번호