Skip to content
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

Jaehyeon week1 #2

Open
wants to merge 16 commits into
base: jaehyeon
Choose a base branch
from
Open

Jaehyeon week1 #2

wants to merge 16 commits into from

Conversation

fanta4715
Copy link
Collaborator

@fanta4715 fanta4715 commented Mar 23, 2024

1주차 👍

fanta4715 and others added 16 commits March 16, 2024 22:43
- 테이블 구조에 맞는 Entity 생성
- BaseTimeEntity 활용해서 생성,수정시간 관리
- OneToMany로 가져오면 페이지가 복잡해져서 이점이 없다고 판단 -> 삭제
- 불필요한 OneToMany 삭제
- 필요한 OneToMany 추가
- Spring Data JPA의 쿼리 메소드 기본 검증
}

public MemberInfoResponse getMemberInfo(Long memberId) {
memberRepository.findById(memberId);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

return 을 하지 않아도 Member가 반환 되나요 ?.?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants