Open
Conversation
jjj0611
requested changes
Nov 29, 2021
jjj0611
left a comment
jjj0611
left a comment
There was a problem hiding this comment.
충현 선생님 리뷰가 늦었네요.
그런데 요구사항이 반영되지 않아서 리뷰를 드리기가 조금 어려운 부분이 있습니다ㅠ
리뷰가 안달리는 동안에 푸시하시면 반영이 되니 구현이 늦어졌고,
리뷰도 늦어지면 그 동안 구현을 더 진행해주시면 좋을 것 같아요!
요구사항에서는 Dao 자체를 모두 JPA를 사용해서 구현해보는 방향에 대해서 말하고 있는데,
기존 Dao 구현체들이 JpaRespository를 사용하도록 공부해보고 적용해보면 어떨까요?
일단 해당 커밋은 request change로 하겠습니다.
|
|
||
| import java.util.Optional; | ||
|
|
||
| public interface DocumentRepository extends JpaRepository<DocumentResponseDto, Long> { |
There was a problem hiding this comment.
JpaRepository 인터페이스의 generic으로 어떤 값들을 원하는지 한 번 학습해보면 좋을 것 같아요!
|
|
||
| @Enumerated(EnumType.STRING) | ||
| private ApprovalState approvalState; | ||
| private Long drafterId; |
There was a problem hiding this comment.
drafter와의 관계가 객체 관계가 아니라 id만 가지고 있는 상태인 것 같은데,
이렇게 구현하신 이유가 있을까요?
| import playground.constants.ApprovalState; | ||
| import playground.constants.Category; | ||
| import learning.constants.ApprovalState; | ||
| import learning.constants.Category; |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
안녕하세요!
3주차 요구사항을 시도해보았는데 완성하지는 못했습니다.
시도해봤지만 실패한 사항들
persistence.xml에 hibernate.hbm2ddl.auto property를 추가했는데 테이블이 생성되지 않음...? :(
어떻게 적용하는지 잘 모르겠습니다
결론적으로는 Document, DocumentApproval, User 클래스에 몇 가지 어노테이션을 추가했습니다.
(Entity, Table, Id 등등...)
시간이 다 되어서 이 상태로 제출합니다.