Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#18 feat: S3 API 구현 #23

Merged
merged 1 commit into from
Aug 10, 2023
Merged

#18 feat: S3 API 구현 #23

merged 1 commit into from
Aug 10, 2023

Conversation

lsn5963
Copy link
Member

@lsn5963 lsn5963 commented Aug 9, 2023

📌 이슈 번호

#18

📢 의논 사항

  • 프로젝트에 맞게 S3를 다시 정리해야할 수도 있습니다.

Copy link
Contributor

@5jisoo 5jisoo left a comment

Choose a reason for hiding this comment

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

와우.. S3 쉽지 않네요 수고하셨습니다👍

Copy link
Contributor

@ri-naa ri-naa 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
Contributor

@JoongHyun-Kim JoongHyun-Kim left a comment

Choose a reason for hiding this comment

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

S3 구현 수고하셨습니다!

;

//https://jforj.tistory.com/261
public List<String> upload(MultipartFile[] multipartFileList) throws IOException {
Copy link
Contributor

Choose a reason for hiding this comment

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

저희는 이미지가 여러장이 아니라 그냥 String으로 반환해도 될 것 같아요!

return imagePathList;
}
}

Copy link
Contributor

Choose a reason for hiding this comment

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

DB에 이미지 url 저장하는 부분이 추가되어야 할 것 같긴 합니다!

@lsn5963 lsn5963 merged commit 733ddfe into develop Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants