Skip to content

Conversation

@starboxxxx
Copy link
Collaborator

📄 요약(Summary)

댓글 작성 API 구현

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • @joowojr @parkjaehak 댓글 작성 시 일반 댓글과 첨부파일을 함께 작성할 수 없기 때문에 일반 댓글 작성 API와 첨부파일 댓글 작성 2개로 나누어 API를 제작하였습니다.

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

스크린샷 2025-01-31 오전 12 25 05 댓글 작성 시 content에 해당 내용 본문을 넣었는데, 댓글 첨부파일 작성시에는 content를 null값으로 하는게 맞을지 아니면 attachment로 통일하는 것이 맞는지 의견을 여쭤보고 싶습니다.

첨부 자료

스크린샷 2025-01-31 오전 12 31 55 @parkjaehak 현재 댓글 및 댓글 첨부파일 작성시 다음과 같이 DB에 값이 저장되고 있는데 이것이 맞는지 확인 부탁드립니다!

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

  • [ ✅ ] PR 양식에 맞게 작성했습니다
  • [ ✅ ] 모든 테스트가 통과했습니다
  • [ ✅ ] 프로그램이 정상적으로 작동합니다
  • [ ✅ ] 적절한 PR 라벨을 설정했습니다
  • [ ✅ ] 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #{이슈 번호}

@starboxxxx starboxxxx self-assigned this Jan 30, 2025
@starboxxxx starboxxxx added ✨ feature 구현·개선 사항에 관련된 내용입니다 MID 우선순위 중 labels Jan 30, 2025
@starboxxxx starboxxxx requested review from joowojr and parkjaehak and removed request for joowojr January 30, 2025 15:35
@parkjaehak
Copy link
Collaborator

  1. 고민 중인 사항
    저는 첨부파일일 경우 content는 null로 하는 것으로 생각하고 작성하였습니다.

  2. 첨부파일
    우선 modifeid_field와 new_value는 더이상 컬럼에 필요없고 대신 modified_status 만 추가되었습니다. 댓글과 첨부파일 생성 시 위와 같은 형태로 taskhistory 테이블에 저장되는 것이 맞습니다. 짤려서 안보이는 것 같은데 comment_id가 fk로 되어있는지 확인한번 부탁드려요!

Comment on lines 37 to 48

// 일반 회원일 경우 => 요청자인지 확인
// 담당자일 경우 => 처리자인지 확인
if ((member.getMemberInfo().getRole() == MemberRole.ROLE_MANAGER)
&& !(member.getMemberId() == task.getProcessor().getMemberId())) {
throw new ApplicationException(MemberErrorCode.NOT_A_COMMENTER);
}

if ((member.getMemberInfo().getRole() == MemberRole.ROLE_USER)
&& !(member.getMemberId() == task.getRequester().getMemberId())) {
throw new ApplicationException(MemberErrorCode.NOT_A_COMMENTER);
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

validation 같은 경우 도메인 내에 작성해주시는 것이 좀더 DDD에 가까울 것 같습니다.

@starboxxxx starboxxxx merged commit 08add89 into develop Jan 31, 2025
@joowojr joowojr deleted the CLAP-177 branch February 3, 2025 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feature 구현·개선 사항에 관련된 내용입니다 MID 우선순위 중

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants