Skip to content

Comments

로고와 피드 이미지 업로드 및 삭제 기능 구현 및 리팩터링#200

Merged
Due-IT merged 2 commits intodevelop/befrom
refactor/#199-refactor_feed_upload
Mar 17, 2025
Merged

로고와 피드 이미지 업로드 및 삭제 기능 구현 및 리팩터링#200
Due-IT merged 2 commits intodevelop/befrom
refactor/#199-refactor_feed_upload

Conversation

@PororoAndFriends
Copy link
Collaborator

@PororoAndFriends PororoAndFriends commented Mar 15, 2025

#️⃣연관된 이슈

ex) #199

📝작업 내용

로고와 피드 이미지 업로드 및 삭제 기능 구현 및 리팩터링

image
  • 로고의 경우 1개의 로고 이미지만 유지되어 새로운 로고가 추가될 경우 이전의 로고는 삭제되도록 구현하였고 별개로 로고의 Delete 요청도 가능하게 구현하였습니다.

  • 이미지 업로드 관련하여 logo와 feed를 분류할 수 있는 FileType Enum을 추가하였습니다.

  • 피드 업로드 기능을 분리하여 구현하였는데, 이는 추후 Signed URL을 통해 프론트 측에서 파일을 직접 클라우드에 올리는 방식으로 수정될 것을 대비하여 미리 분리해 두었습니다.
  • 현재는 MultipartFile을 받아 백엔드 서버에서 업로드해 주는 방식으로 구현하여 정상적으로 동작되도록 하였습니다.
  • 또한 현재는 Post 메서드로 구현되어 있지만 Signed URL이 구현될 경우 Get 메서드로 수정될 예정입니다.
  • 추후 Signed URL 작업이 필요한 곳은 TODO로 표시해 두었습니다.
image
  • 피드 이미지는 Post 메서드로 통일하여 저장하도록 하였습니다.
  • post 요청시 저장되어 있는 정보와 새로 들어온 정보를 비교하여 없어진 데이터를 자동으로 삭제 요청하도록 수정하였습니다.
  • 이를 통해 사진을 삭제하고 싶다면 별다른 Delete 요청이 아닌 Post 요청으로 삭제하고 싶은 URL만 제외하고 보내면 자동으로 삭제되도록 구현되었습니다.
image

@PororoAndFriends PororoAndFriends added 🔨 Refactor 코드 리팩토링 💾 BE Backend labels Mar 15, 2025
@PororoAndFriends PororoAndFriends self-assigned this Mar 15, 2025
@github-actions
Copy link

github-actions bot commented Mar 15, 2025

Test Results

1 tests   1 ✅  0s ⏱️
1 suites  0 💤
1 files    0 ❌

Results for commit e50132e.

♻️ This comment has been updated with latest results.

@Zepelown Zepelown requested review from Due-IT and Zepelown and removed request for Zepelown March 16, 2025 04:46
Copy link
Member

@Zepelown Zepelown left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

Copy link
Member

@Zepelown Zepelown left a comment

Choose a reason for hiding this comment

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

update 기능이 더 깔삼해졌네요
고생하셨습니다

Copy link
Collaborator

@Due-IT Due-IT left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 의도한 대로 잘 동작할것 같네요

@Due-IT Due-IT merged commit ad76b0f into develop/be Mar 17, 2025
2 checks passed
@PororoAndFriends PororoAndFriends deleted the refactor/#199-refactor_feed_upload branch April 3, 2025 00:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💾 BE Backend 🔨 Refactor 코드 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants