Skip to content

Conversation

@parkjaehak
Copy link
Collaborator

@parkjaehak parkjaehak commented Jan 30, 2025

📄 요약(Summary)

작업 승인, 작업 처리자 변경, 작업 상태 변경시 작업 히스토리를 생성한다.

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  1. 아래 사진과 같이 변경된 상태를 저장하기 위한 modifeidStatus 추가
    -> 기존 task를 참조하여 status를 가져오는 것은 task의 상태가 변하게 되면 함께 변하기 때문에 잘못된 코드
    -> 따라서 시에나 말씀대로 상태의 대한 값을 저장할 수 있는 modifiedStatus 추가
    -> modifiedMember도 작업 처리자 변경 시점에 닉네임을 가져와 저장할 수 있지만 닉네임 변경에 대한 부분 고려하여 id를 참조하도록 유지

  2. ChangeTaskController 내의 작업 상태 변경, 작업 처리자 변경, 작업 승인 API 호출 시 그에 맞는 작업 히스토리 생성하도록 설정

  3. 댓글 추가, 댓글 첨부파일 추가 시 작업 히스토리 생성 필요

  4. 댓글 삭제가 가능하다면 댓글 삭제 시점에 히스토리를 삭제하거나 히스토리 조회시 삭제된 댓글은 히스토리로 조회하지 않는 로직 설정 필요

Requirements for Reviewer

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

PR Log

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

새롭게 배운 것

고민 중인 사항

첨부 자료

image

Requirements for Reviewer

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

✅ 체크리스트(Checklist)

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

🚪 이슈 번호(Issue numbers)

Closes #168

@parkjaehak parkjaehak added 🧽 cleanup 코드를 더 깔끔하게 만들기만 하고, 코드 작동 방식이나 출력에 대한 부분을 변경하지 않습니다. ✨ feature 구현·개선 사항에 관련된 내용입니다 ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다. HIGH 우선순위 상 labels Jan 30, 2025
@parkjaehak parkjaehak self-assigned this Jan 30, 2025
@parkjaehak parkjaehak requested review from Sihun23 and nano-mm and removed request for Sihun23 and nano-mm January 30, 2025 03:42
@starboxxxx starboxxxx requested review from starboxxxx and removed request for nano-mm January 30, 2025 10:02
Copy link
Collaborator

@starboxxxx starboxxxx left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

@parkjaehak parkjaehak merged commit 22640a6 into develop Jan 30, 2025
1 check passed
@parkjaehak parkjaehak deleted the CLAP-176 branch February 3, 2025 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧽 cleanup 코드를 더 깔끔하게 만들기만 하고, 코드 작동 방식이나 출력에 대한 부분을 변경하지 않습니다. ✨ feature 구현·개선 사항에 관련된 내용입니다 HIGH 우선순위 상 ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-176 작업 승인, 상태변경, 처리자 변경 시 작업 히스토리 생성

3 participants