Skip to content

Conversation

@EunjungOh123
Copy link
Collaborator

@EunjungOh123 EunjungOh123 commented Dec 3, 2022

No description provided.

@younghoondoodoom
Copy link
Contributor

There is no coverage information present for the Files changed

Total Project Coverage 25.11%

@EunjungOh123 EunjungOh123 linked an issue Dec 3, 2022 that may be closed by this pull request
4 tasks
@naekang
Copy link
Collaborator

naekang commented Dec 4, 2022

깔끔하네요! 수고하셨습니다 🤟🤟🤟

@naekang naekang added the feature New feature or request label Dec 4, 2022
Copy link
Collaborator

@bokyung95 bokyung95 left a comment

Choose a reason for hiding this comment

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

도메인 작성하시느라 고생많으셨습니다!
매핑 관련하여 궁금한 점이 있어서 리뷰 남깁니다.
Comment, Heart, Post 엔티티는 Member와 양방향으로 하지 않아도 문제가 없을까욤?

Copy link
Contributor

@younghoondoodoom younghoondoodoom left a comment

Choose a reason for hiding this comment

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

깔끔하네요! 고생하셨습니다~


@Id
@GeneratedValue(strategy = GenerationType.IDENTITY)
@Column(name = "image_id")
Copy link
Contributor

Choose a reason for hiding this comment

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

도메인 이름 때문에 post_image_id는 어떠신가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

넵 수정했습니다~!

@EunjungOh123
Copy link
Collaborator Author

EunjungOh123 commented Dec 5, 2022

도메인 작성하시느라 고생많으셨습니다! 매핑 관련하여 궁금한 점이 있어서 리뷰 남깁니다. Comment, Heart, Post 엔티티는 Member와 양방향으로 하지 않아도 문제가 없을까욤?

member 쪽 엔티티가 많은 테이블과 연관 관계 맺으면 클래스가 복잡해지는 것 같아서 일단 단방향으로 설정하고 나중에 필요해지면 추가할 생각입니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

게시판 도메인 작성

5 participants