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.

Sign up for GitHub

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

Jump to bottom

[randy] Chapter 7. JPA를 통한 엔티티 설계, 매핑 & 프로젝트 파일 구조 이해 #27

Merged
merged 4 commits into from
May 30, 2024
Merged

[randy] Chapter 7. JPA를 통한 엔티티 설계, 매핑 & 프로젝트 파일 구조 이해 #27

merged 4 commits into from
May 30, 2024

Conversation

Jangwon0319
Copy link

@Jangwon0319 Jangwon0319 commented May 29, 2024 •
edited
Loading

#️⃣ 연관된 이슈

#26


✅ 워크북 체크리스트

  • 모든 핵심 키워드 정리를 마쳤나요?
  • 핵심 키워드에 대해 완벽히 이해하셨나요?
  • 이론 학습 이후 직접 실습을 해보는 시간을 가졌나요?
  • 미션을 수행하셨나요?
  • 미션을 기록하셨나요?

✅ 컨벤션 체크리스트

  • 디렉토리 구조 컨벤션을 잘 지켰나요?
  • pr 제목을 컨벤션에 맞게 작성하였나요?
  • pr에 해당되는 이슈를 연결하였나요?
  • 적절한 라벨을 설정하였나요?
  • 스터디원들에게 code review를 요청하기 위해 reviewer를 등록하였나요?
  • 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?

📌 주안점

@Jangwon0319 Jangwon0319 added the ✨ Chapter07 챕터 7 label May 29, 2024
@Jangwon0319 Jangwon0319 requested review from ls-rain and KkomSang May 29, 2024 13:45
@Jangwon0319 Jangwon0319 self-assigned this May 29, 2024
@Jangwon0319 Jangwon0319 changed the base branch from main to randy/main May 29, 2024 13:46
KkomSang
KkomSang approved these changes May 30, 2024
View reviewed changes
Copy link

@KkomSang KkomSang left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!! 기말고사 화이팅🤓

docs/chapter7/Ch07Keyword.md
//em.update(member) 이런 코드가 있어야 하지 않을까?

transaction.commit(); // 트랜잭션 커밋
```
Copy link

Choose a reason for hiding this comment

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

예시가 있으니까 더 잘 이해되네용!!

ls-rain
ls-rain approved these changes May 30, 2024
View reviewed changes
Copy link

@ls-rain ls-rain left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

docs/chapter7/Ch07Keyword.md

<br>

- **영속 엔티티의 동일성 보장**
Copy link

Choose a reason for hiding this comment

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

상세한 예시까지 들어주셔서 이해하기 좋습니다. 감사합니다!!

@Jangwon0319 Jangwon0319 merged commit 6a2b13b into randy/main May 30, 2024
@Jangwon0319 Jangwon0319 deleted the randy/#7 branch May 30, 2024 02:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ls-rain ls-rain ls-rain approved these changes

@KkomSang KkomSang KkomSang approved these changes

Assignees

@Jangwon0319 Jangwon0319

Labels
✨ Chapter07 챕터 7
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Jangwon0319 @ls-rain @KkomSang