Skip to content

Conversation

@starboxxxx
Copy link
Collaborator

📄 요약(Summary)

작업 상태를 변경할 수 있는 API 구현

✍🏼 상세(More)

PR Desciption

  • 작업 상태가 변경되면 해당 사항에 대한 알림 또한 생성되는데 이는 HIGH 기능들 먼저 구현한 후 추가 하려고 계획하고 있습니다.

Requirements for Reviewer

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

PR Log

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

새롭게 배운 것

고민 중인 사항

첨부 자료

Requirements for Reviewer

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

✅ 체크리스트(Checklist)

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

🚪 이슈 번호(Issue numbers)

Closes #{이슈 번호}

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

@joowojr joowojr left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!
지금 updatetaskusecase에서 너무 많은 컴포넌트들에 의존하고 있어서
구분/ 상태 변경의 유스케이스는 같이 따로 빼면 좋을것 같습니다!
변경하고 머지하셔도 될것같아요:)

@starboxxxx starboxxxx merged commit b8a7a09 into develop Jan 27, 2025
@starboxxxx starboxxxx deleted the CLAP-154 branch January 27, 2025 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants