Skip to content

Conversation

@joowojr
Copy link
Contributor

@joowojr joowojr commented Jan 26, 2025

📄 요약(Summary)

요청 생성 및 수정에 첨부파일 업로드 로직 추가

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • 첨부파일 업로드 방식 변경 : presigned url -> multi part

    • 카카오 클라우드에서 presigned url 방식을 제공하지 않아 멀티 파트 형식으로 구현하게 되었습니다.
    • 방식 변경에 따라 request dto 스펙을 수정하였습니다.
  • S3 업로드 서비스 구현

    • 현재 파일 배열을 업로드 하는 서비스와, 단일 파일을 업로드하는 서비스 메서드를 모두 구현하였습니다.
    • 추후에 댓글을 통해 첨부파일을 업로드할 때 사용해주시면 됩니다. 댓글 기능 구현은 제일 후순위로 두어야 할것 같습니다.
  • 요청 usecase 리팩토링

    • 요청 생성/ 수정 시에 첨부파일과 알림을 핸들링하는 부분을 메서드로 분리하여 책임을 분리하고 가독성을 개선시켰습니다.

Requirements for Reviewer

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

PR Log

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

새롭게 배운 것

고민 중인 사항

첨부 자료

Requirements for Reviewer

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

@hyoseong-Choi s3 설정에 관한 환경 변수를 vm에 등록해놓은 상태입니다. 참고 부탁드립니다.

✅ 체크리스트(Checklist)

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

🚪 이슈 번호(Issue numbers)

Closes #76

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

Choose a reason for hiding this comment

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

@value 쓰는 부분이 test의 yml파일에 없어서 Placeholder에러가 생기는거 같은데 이것만 수정해주시면 좋을 것 같습니다

<footer>
- 관련: #76
@joowojr joowojr merged commit b03952a into develop Jan 26, 2025
1 check passed
@joowojr joowojr deleted the CLAP-113 branch January 26, 2025 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feature 구현·개선 사항에 관련된 내용입니다 HIGH 우선순위 상 ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-113 오브젝트 스토리지 첨부파일 업로드 로직 구현

4 participants