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

#74 Feat: add post, comment, postimage domain #75

Merged
merged 6 commits into from
Nov 5, 2024
Merged

Conversation

kjiyun
Copy link
Collaborator

@kjiyun kjiyun commented Oct 8, 2024

#️⃣연관된 이슈

ex) #74

📝작업 내용

게시판, 댓글, 게시판 이미지 도메인 생성

스크린샷 (선택)

💬리뷰 요구사항(선택)

대댓글, 댓글 작성자 추가 필요

@kjiyun kjiyun requested review from yumzen and boogiewooki02 October 8, 2024 11:32
@yumzen yumzen added the ✨ Feature 기능 개발 label Oct 9, 2024
@yumzen yumzen requested review from woogieon8on and removed request for boogiewooki02 October 9, 2024 04:50
Copy link
Collaborator

Choose a reason for hiding this comment

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

insta_clone
UMC에서 인스타 클론코딩 스터디하면서 작성한 ERD입니다
댓글, 좋아요 엔티티 구현할 때 참고하세욥

댓글, 대댓글 엔티티를 따로 두는 것보다 지은이 말처럼 댓글 엔티티 하나로 관리하는 방법이 디비 구조가 간결해져서 좋을 것 같습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

감사합니다 !!

Copy link
Member

@yumzen yumzen left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!!!!!!!!!!!!!!!!!!!😄😄😄😄😄😄😄😄😄

}

public static List<PostImage> toPostImage(List<PostImageCreateRequest> postImageCreateRequests, Post post) {
return Optional.ofNullable(postImageCreateRequests)
Copy link
Collaborator

Choose a reason for hiding this comment

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

null 여부를 컨버터에서 검증하니까 서비스 로직이 훨씬 깔끔해지네요!! 굿 굿

@kjiyun kjiyun merged commit af199b2 into kahluaband:main Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants