-
Notifications
You must be signed in to change notification settings - Fork 200
[5기 김남규, 박유진] Spring Boot JPA 게시판 구현 미션 제출합니다. #269
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
Open
eugene225
wants to merge
41
commits into
prgrms-be-devcourse:eugene_namgyu
Choose a base branch
from
eugene225:main
base: eugene_namgyu
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
5d1b20c
init commit
eugene225 e67814f
Chore : 프로젝트 환경설정
eugene225 71f3790
Feat : entity 생성 - User, Post
eugene225 3bc557b
Fix : 연관관계 매핑 오류 해결
GiHoo b0c5ecb
Feat : Post 관련 Dto 구현
GiHoo 426d642
Feat : PostRepository 구현
GiHoo 7fbe52e
Refactor : 사용하지 않는 애노테이션 제거
GiHoo 91b8736
Feat : UserRepository 구현
GiHoo ec97bec
Feat : UserDto 구현
GiHoo 5f485c8
Feat : PostService 구현
GiHoo 462b862
Feat : PostController 구현
GiHoo 1c49847
Refactor : PostDto에 UserDto 추가
GiHoo bc065c9
Feat : setUser, update 메서드 추가
GiHoo f4d7a9b
Feat : Converter 구현
GiHoo afa8a7d
Refactor : update Api 수정
GiHoo 8bfc7d2
Test : PostService 테스트
GiHoo ff17326
Feat : ApiReponse 생성 및 postController 적용
eugene225 95e91f8
Chore : restdocs 플러그인 추가
eugene225 d15af2a
Test : PostController 테스트 및 문서화 작업
eugene225 33b67e2
Feat : GlobalExceptionHandler 구현
eugene225 c008314
Docs : api 문서화
eugene225 268882d
Feat : postController ApiResponse 적용
eugene225 cc0e870
Test : findById 실패 테스트 추가
GiHoo 63ed5a5
1차 피드백 수정
GiHoo 35628e6
1차 피드백 수정
GiHoo b01f6ed
Refactor: HTTP 메서드 변경 (POST -> PUT)
GiHoo 57127d9
Refactor: create를 위한 로직 수정(dto 생성 및 converter 추가)
GiHoo 4cafb55
Refactor: update를 위한 로직 수정(null 체크)
GiHoo f6a301b
Refactor: update를 위한 로직 수정
GiHoo 0b112f1
Remove: ApiResponse 제거
GiHoo 0f91afa
Refactor : ApiResponse -> ResponseEntity 변경
GiHoo 6599c3c
Refactor : @RequiredArgsConstructor 로 변경
GiHoo bdda4e9
Test: 변경한 Dto 반영
GiHoo 055fdd0
Test: 변경한 Dto 반영
GiHoo 71d2f67
Docs: Test를 위한 환경 설정 추가
GiHoo 226bcac
Refactor: BaseEntity abstract class 변경
GiHoo 2068b99
Feat: CustomException 생성
eugene225 100260e
Refactor: CustomException 적용
eugene225 e4bc9c1
Refactor: CustomException 적용
eugene225 bd2b1ca
Refactor: update 로직 변경
eugene225 858c1ac
Comment: 1차 코드리뷰 주석 삭제
eugene225 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 hidden or 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.
관련 검증을 Age 자체에 넘길 수 있도록 값객체로 선언할 수 있을 것 같아요.
나중에 고민해보시면 좋을 것 같아요 👍