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

feat: retrospect, comment 구현 #26

Open
wants to merge 13 commits into
base: develop
Choose a base branch
from
Open

feat: retrospect, comment 구현 #26

wants to merge 13 commits into from

Conversation

ybkang1108
Copy link
Collaborator

@ybkang1108 ybkang1108 commented Feb 10, 2025 •
edited
Loading

@ybkang1108
feat: 회고 작성, 수정 구현
6fef94b 
- round의 application에 RetrospectService 추가
- RoundMemberRepository 추가
@ybkang1108
feat: 회고 삭제, 조회 기능 구현
27da524 
- 회고 삭제
- 개별 회고 조회
- 모든 회고 조회

기능 구현
@ybkang1108
refactor: response 수정
19e8bf8 
api명세서에 맞게 response 수정
@ybkang1108
feat: comment 작성, 수정, 조회, 삭제 구현 및 Controller 분리
a961cc6 
- RoundController에서 Comment와 Retrospect 분리
- Comment를 save하기 위한 CustomRepository 추가
- 기능 구현
@ybkang1108
feat: 댓글, 회고 기능 구현
bd53c9d 
- 댓글 생성, 수정, 삭제, 조회
- 회고 생성, 수정, 삭제, 조회
@ybkang1108
fix: 예외 처리 추가
142b0d2 
- service에서 throw CustomException 추가
- controller에 try catch 추가
- 예외처리 구체화 필요..
@ybkang1108
chore: todo 추가
aa1b78e
@ybkang1108
refactor: try catch 제거
9147066
@cmj7271
Copy link
Collaborator

cmj7271 commented Feb 18, 2025

comment 관련 레포지토리, 서비스, api 를 comment 도메인으로 옮기는 편이 좋을 거 같습니다.
제가 리뷰 마저 하고 반영해서 커밋하겠습니다.

cmj7271
cmj7271 requested changes Feb 19, 2025
View reviewed changes
Copy link
Collaborator

@cmj7271 cmj7271 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

리뷰 올렸습니다
추가로, repository 는 변동사항 좀 있을거 같아서 유심히 봐주세요

src/main/java/com/gdgoc/study_group/round/dto/CommentDTO.java Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

요거 제가 노션에 잘못 적은거 같은게, member_id 도 있어야할 거 같아요.
API 명세서에 반영할테니까 수정 부탁드립니다

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

혹시 댓글 작성/수정도 member_id 필요한가요??

@ybkang1108
refactor: round custom repository 제거
fbaca1c 
- custom repository 제거
- @query를 사용하여 Round Member저장하도록 변경
@ybkang1108
refactor: retrospect update 함수 이름 변경
8644f89
@ybkang1108
refactor: 회고 작성/삭제 반환값 제거
b7d95db
@ybkang1108
refactor: comment 이동
3148c44 
- allargs -> public, create제거
- comment를 round에서 comment로 이동
- comment dto: response와 request 분리
- comment response에는 member_id 추가
@ybkang1108
chore: 불필요한 부분 제거
434a79d 
- 사용하지 않는 import제거
- retrospect에서 validate 함수 제거
- json property로 필드값 변경
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cmj7271 cmj7271 cmj7271 requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

comment domain api 제작(2/9까지) retrospect domain api 제작(2/9까지)
2 participants
@ybkang1108 @cmj7271