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

Feat : 이미지 추가 및 삭제 api 구현, 게시물 작성 및 수정 기능 구현 #29

Merged
merged 12 commits into from
Jan 24, 2025

Conversation

chanmin-00
Copy link
Member

#️⃣연관된 이슈

Close #27

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

S3 설정

이미지 추가 및 삭제 api 구현

image
  • 게시물 등록하기 api를 요청하기 전, 이미지 추가 api 실행
  • 이미지 추가 api를 실행하면 S3로 이미지를 업로드 하고, imageId 반환
  • 반환된 이미지 id들은 추후, 게시물 작성 api의 입력값으로 사용되어야 함

게시물 작성 api 구현

image
  • 제목, 내용, 타입, 이미지 id 목록을 입력을 받아서 게시물 작성

게시물 수정 api 구현

image
  • 이 api를 호출하기 전에 사진 수정이 필요할 경우, 필요하지 않은 사진은 삭제 api를 통해서 삭제, 추가할 사진은 이미지 추가 api를 통해서 추가
  • 입력으로 넘겨줘야 할 imageId에는 새로 추가한 이미지의 id만 넘겨주면 됨

💬리뷰 요구사항(선택)

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

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@chanmin-00 chanmin-00 added the ✨ Feat 새로운 기능 추가 label Jan 20, 2025
@chanmin-00 chanmin-00 requested a review from chaen-ing January 20, 2025 13:58
@chanmin-00 chanmin-00 self-assigned this Jan 20, 2025
Copy link
Member

@chaen-ing chaen-ing left a comment

Choose a reason for hiding this comment

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

확인했습니다!!💯

String fileName = buildFileName(folderName, file.getName());
String uploadUrl = uploadToS3(file, fileName);

System.out.println("uploadUrl: " + uploadUrl);
Copy link
Member

Choose a reason for hiding this comment

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

log.info와 System.out.println을 구분해서 사용하신는 것 같은데 이유가 있을까욥??

Copy link
Member Author

Choose a reason for hiding this comment

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

아 제가 오류가 나서 확인해본다고 써놓은건데 삭제를 깜박했네요. 지우도록 할게요

Copy link
Member

Choose a reason for hiding this comment

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

넵!! 머지하셔도 될거같습니다~~!👍

Copy link
Member Author

Choose a reason for hiding this comment

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

삭제했습니다 머지하겠습니다

@chanmin-00 chanmin-00 changed the title Feat/image #27 Feat : 이미지 추가 및 삭제 api 구현, 게시물 작성 및 수정 기능 구현 Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feat 새로운 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 게시물 작성 및 이미지 업로드 기능을 구현한다.
2 participants