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

[고도화] 댓글 기능 고도화 - 1차 대댓글: 도메인 업데이트 #67

Closed
Tracked by #66
lmw7414 opened this issue Jan 8, 2023 · 0 comments
Closed
Tracked by #66

[고도화] 댓글 기능 고도화 - 1차 대댓글: 도메인 업데이트 #67

lmw7414 opened this issue Jan 8, 2023 · 0 comments
Assignees
@lmw7414
Labels
enhancement New feature or request
Milestone
게시판 서비스 만들기

Comments

@lmw7414
Copy link
Owner

lmw7414 commented Jan 8, 2023 •
edited
Loading

대댓글 기능을 위한 도메인 정보를 업데이트 한다.

  • ArticleComment 에 parentCommentId 추가
@lmw7414 lmw7414 mentioned this issue Jan 8, 2023
Closed
4 tasks
@lmw7414 lmw7414 self-assigned this Jan 8, 2023
@lmw7414 lmw7414 added the enhancement New feature or request label Jan 8, 2023
@lmw7414 lmw7414 added this to the 게시판 서비스 만들기 milestone Jan 8, 2023
@lmw7414 lmw7414 changed the title 도메인 업데이트 [고도화] 댓글 기능 고도화 - 1차 대댓글 - 도메인 업데이트 Jan 8, 2023
@lmw7414 lmw7414 changed the title [고도화] 댓글 기능 고도화 - 1차 대댓글 - 도메인 업데이트 [고도화] 댓글 기능 고도화 - 1차 대댓글: 도메인 업데이트 Jan 8, 2023
lmw7414 added a commit that referenced this issue Jan 8, 2023
@lmw7414
#67 - 대댓글 도메인 설계
9d4174f 
대댓글 도메인 안에서 부모, 자식 관계를 설정하는 코드를 추가. 자식 댓글의 컬렉션 변화가 쿼리에 반영되게끔 cascading 규칙을 모두 적용
이번엔 단방향 연관관계 설정을 사용해보기로 함.
따라서 부모 댓들은 엔티티가 아닌 `Long` id를 직접 표현

또한 자식 댓글을 추가할 수 있는 메소드 추가 제공
@lmw7414 lmw7414 mentioned this issue Jan 8, 2023
Merged
lmw7414 added a commit that referenced this issue Jan 8, 2023
@lmw7414
Merge pull request #71 from lmw7414/feature/#67-child-comments-domain
96379e2 
[고도화] 댓글 기능 고도화 - 1차 대댓글: 도메인 업데이트
@lmw7414 lmw7414 closed this as completed Jan 8, 2023
oyoungsun added a commit to oyoungsun/fastcampus-project-board-1 that referenced this issue Jan 9, 2023
@oyoungsun
Merge pull request #1 from oyoungsun/feature/lmw7414#67-usecase-update
81a4ee6 
lmw7414#67-2 usecaese update
@oyoungsun oyoungsun mentioned this issue Jan 9, 2023
Merged
oyoungsun added a commit to oyoungsun/fastcampus-project-board-1 that referenced this issue Jan 9, 2023
@oyoungsun
Merge pull request #2 from oyoungsun/feature/lmw7414#68-new-doamin
551d1b3 
Merge pull request #1 from oyoungsun/feature/lmw7414#67-usecase-update
oyoungsun pushed a commit to oyoungsun/fastcampus-project-board-1 that referenced this issue Jan 17, 2023
lmw7414#67-2 usecaese update
670ca84
oyoungsun added a commit to oyoungsun/fastcampus-project-board-1 that referenced this issue Jan 17, 2023
@oyoungsun
Merge pull request #1 from oyoungsun/feature/lmw7414#67-usecase-update
365e4af 
lmw7414#67-2 usecaese update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lmw7414 lmw7414

Labels
enhancement New feature or request
Projects
None yet
Milestone
게시판 서비스 만들기
Development

No branches or pull requests
1 participant
@lmw7414