Skip to content

Conversation

@parkjaehak
Copy link
Collaborator

@parkjaehak parkjaehak commented Feb 12, 2025

📄 요약(Summary)

작업 요청 파일 업로드 오류

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • 삭제할 첨부파일이 없으면 첨부파일을 저장하는 로직이 없기 때문에 이 부분을 분기처리하여 추가하였습니다.

Requirements for Reviewer

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

PR Log

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

새롭게 배운 것

고민 중인 사항

첨부 자료

       if (!request.attachmentsToDelete().isEmpty()) {
            updateAttachments(request.attachmentsToDelete(), files, task);
        }
        else {
            updateAttachmentWithoutAttachmentsToDelete(files, task);
        }

Requirements for Reviewer

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

✅ 체크리스트(Checklist)

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

🚪 이슈 번호(Issue numbers)

Closes #511

@parkjaehak parkjaehak added HIGH 우선순위 상 🚑 hotfix labels Feb 12, 2025
@parkjaehak parkjaehak self-assigned this Feb 12, 2025
@parkjaehak parkjaehak force-pushed the CLAP-399 branch 2 times, most recently from 5ff6094 to 47a5349 Compare February 12, 2025 06:06
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.

수고하셨습니다!

@parkjaehak parkjaehak merged commit 3e2f124 into develop Feb 12, 2025
1 check passed
@joowojr joowojr deleted the CLAP-399 branch February 12, 2025 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

HIGH 우선순위 상 🚑 hotfix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-399 작업 수정시 파일 업로드 오류

3 participants