-
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 구현 #26 #42
Merged
Merged
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
295c97c
feat: 특정 방문 기록 삭제 API 구현
devhoya97 5eefdf6
feat: 양수가 아닌 id로 특정 방문 기록 삭제를 시도할 때 예외 처리 기능 구현
devhoya97 e03d7d5
feat: 방문 기록 삭제 시 방문 로그도 함께 삭제되는 기능 구현
devhoya97 30a2fc3
refactor: 커스텀 예외를 제거하는 방향으로 변경
devhoya97 8b9914c
fix: 예외를 못 잡던 문제 해결
devhoya97 2748c25
refactor: 메서드명 적절하게 변경
devhoya97 939d46d
build: Docker Compose Setting #27 (#40)
Ho-Tea b735d14
feat: 여행 상세 생성 API 구현 #18 (#43)
linirini 60171c8
build: CD yml 파일 구성 #28 (#53)
Ho-Tea 3bf8c9b
fix: rebase 과정에서 파일이 꼬인 문제 해결
devhoya97 495f98e
test: HttpHeaders.AUTHORIZATION 사용
devhoya97 c2bfc63
refactor: 중간 테이블 엔티티 수정 #56 (#57)
linirini c490e2c
feat: Pin, Visit, VisitLog 생성자에 builder 추가
devhoya97 6a94854
feat: Pin repository 추가
devhoya97 f144a77
refactor: visit이 삭제되기 전에 visit에 의존하는 visitLog들이 먼저 삭제되도록 순서 변경
devhoya97 076e570
test: 방문 기록 삭제에 대한 서비스 슬라이스 테스트 추가
devhoya97 a559d68
test: 방문 기록이 갖는 모든 방문 로그 삭제 메서드 테스트
devhoya97 665ea5f
fix: Modifying을 사용할 때 영속성컨텍스트와 관련하여 발생하던 문제 해결
devhoya97 55218fd
refactor: visitLog의 content를 필수값으로 변경
devhoya97 94d2664
test: 컨벤션에 맞게 Controller 테스트 클래스 변경
devhoya97 fff5c00
fix: ConstraintViolationException의 예외 메시지를 정해둔 형식에 맞게 변경
devhoya97 4ad9148
Merge branch 'develop-be' into feature/#26-delete-visit
devhoya97 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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.
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.
2차 스프린트 개발 범위는 아니지만, content도 필수값이 될 것 같습니다. NotNull 어노테이션을 추가하면 좋을 것 같네요!
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.
간단한 내용이라 반영 했습니다 :)