Skip to content

Comments

[fedragon] Chapter 4. JPA 기초 및 프로젝트 구조#8

Merged
YounchanHa merged 1 commit intofedragon/mainfrom
fedragon/#4
Oct 28, 2025
Merged

[fedragon] Chapter 4. JPA 기초 및 프로젝트 구조#8
YounchanHa merged 1 commit intofedragon/mainfrom
fedragon/#4

Conversation

@YounchanHa
Copy link

@YounchanHa YounchanHa commented Oct 12, 2025

✅ 워크북 체크리스트

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

✅ 컨벤션 체크리스트

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

📌 주안점

@YounchanHa YounchanHa requested review from a team and 2ghrms October 12, 2025 11:26
@YounchanHa YounchanHa self-assigned this Oct 12, 2025
@YounchanHa YounchanHa added the ✨ Chapter04 챕터 4 label Oct 12, 2025
Copy link

@woong-ja woong-ja left a comment

Choose a reason for hiding this comment

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

1:N 연관관계 매핑이 안되어 있어요 (@onetomany)

Choose a reason for hiding this comment

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

1:N 연관관계 매핑이 안되어 있어요

Choose a reason for hiding this comment

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

Choose a reason for hiding this comment

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

1:N 연관관계 매핑이 안되어 있어요

Choose a reason for hiding this comment

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

Choose a reason for hiding this comment

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

1:N 연관관계 매핑이 안되어 있어요

Choose a reason for hiding this comment

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

Choose a reason for hiding this comment

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

1:N 연관관계 매핑이 안되어 있어요

Choose a reason for hiding this comment

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

Choose a reason for hiding this comment

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

1:N 연관관계 매핑이 안되어 있어요
@onetomany

Choose a reason for hiding this comment

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

1:N 연관관계 매핑이 안되어 있어요
https://github.com/onetomany

private LocalDate birth;

@Column(length = 50)
private String email;
Copy link
Member

Choose a reason for hiding this comment

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

이메일 주소의 length 는 255로 하는것이 좋을거같아요. 50자를 넘는 이메일도 존재할 수 있어요.
https://stackoverflow.com/questions/386294/what-is-the-maximum-length-of-a-valid-email-address

Copy link
Author

Choose a reason for hiding this comment

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

그러네요 감사합니다

@2ghrms 2ghrms changed the title Feat: 4주차 미션 - ERD 기반 엔티티 생성 및 연관관계 설정 [fedragon] Chapter 4. JPA 기초 및 프로젝트 구조 Oct 28, 2025
@YounchanHa YounchanHa merged commit ebb7f2d into fedragon/main Oct 28, 2025
seorin05 pushed a commit that referenced this pull request Nov 16, 2025
AlphaBs added a commit that referenced this pull request Nov 18, 2025
AlphaBs added a commit that referenced this pull request Nov 18, 2025
AlphaBs added a commit that referenced this pull request Nov 18, 2025
AlphaBs added a commit that referenced this pull request Nov 18, 2025
AlphaBs added a commit that referenced this pull request Nov 18, 2025
woong-ja added a commit that referenced this pull request Nov 18, 2025
woong-ja added a commit that referenced this pull request Nov 19, 2025
AlphaBs added a commit that referenced this pull request Nov 25, 2025
* mission/#8 - 회원가입

* mission/#8 - 가게에 리뷰 추가하기

* mission/#8 - 특정 가게에 가게 추가하기

* mission/#8 - 가게에 미션 추가하기 API

* mission/#8 - 미션시작하기 API
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants