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.

Sign up for GitHub

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

Jump to bottom

feat: 일기 조회 로직 이미지 자체를 전달하도록 변경 #26

Merged
merged 5 commits into from
Nov 26, 2023
Merged

feat: 일기 조회 로직 이미지 자체를 전달하도록 변경 #26

merged 5 commits into from
Nov 26, 2023

Conversation

0703kyj
Copy link
Contributor

@0703kyj 0703kyj commented Nov 25, 2023

개요

  • 일기 조회 로직 이미지 자체를 전달하도록 변경

작업사항

  • DiaryService의 findOne 메서드를 Response 반환에서 Diary자체 반환으로 변경했습니다.
  • PresignedUrlService에서 S3 버킷의 이미지 자체를 가져오는 메서드 추가했습니다.
  • 일기 조회 api를 이미지 url전달에서 이미지 자체를 받아오는 방식으로 수정하였습니다.
  • 일기 전체 조회 api는 이미지 Url에서 일기 id를 받아오도록 수정했습니다.

주의사항

  • api 테스트 결과 사진은 잘 받아와지는 것 확인했습니다.

@0703kyj 0703kyj added the ✏️ API 변경 API 변경 label Nov 25, 2023
@0703kyj 0703kyj requested a review from Ji-soo708 November 25, 2023 14:32
@0703kyj 0703kyj self-assigned this Nov 25, 2023
@Ji-soo708 Ji-soo708 merged commit ff677c3 into main Nov 26, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ji-soo708 Ji-soo708 Ji-soo708 approved these changes

Assignees

@0703kyj 0703kyj

Labels
✏️ API 변경 API 변경
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@0703kyj @Ji-soo708