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 ] 명함 사진 Presigned URL 연결 #323

Merged
merged 6 commits into from
Oct 30, 2024

Conversation

lydiacho
Copy link
Collaborator

@lydiacho lydiacho commented Oct 30, 2024

#️⃣ Related Issue

Closes #316

✅ Done Task

  • api/v1/image/buissness API 연결
  • 기존 프로필사진 API 도 path 바꿔주기 (api/v1/image/profile)

💎 PR Point

지난번 프로필 이미지 업로드에 PresignedURL 연결했던 작업 참고하면서 동일하게 구현했어요!!
그래서 특별히 새로운 내용은 없습니당

기존에 회원가입 APi requestBody에 businessCard 필드 더미주소로 넣어주던 것도 s3에 저장된 파일명 넣어주도록 수정해주었고,
presignedURL을 두개 쓰게 되면서 기존에 profileImage 업로드 관련 함수들 네이밍도 살짝식 수정해서 구분되게 바꿔주었어요!

로직 순서

  1. 처음에 명함인증 뷰에 도착하면 /image/businesscard API에서 presignedURL 주소를 GET 해와요
  2. 명함을 업로드한 직후에는 OCR API 통신(POST)만 진행돼요! 원래는 명함 이미지 업로드 로직도 이 타이밍에 같이 진행됐는데, 명함 인식에 실패할 경우 어차피 다시 촬영을 해야하기 때문에 S3에 실패한 이미지를 올리는건 불필요한 것 같아서 빼줬어요.
  3. 그리고 presignedURL을 통해 S3에 명함 이미지를 업로드하는건 (PUT method) 프로필이미지랑 동일하게 하단 CTA 클릭 시 mutate 실행해줍니다!

📸 Screenshot

위 로직대로 진행되는거 녹화했습니당

i.e.e.e.2024-10-30.i.i.7.13.34.mov

Join request body에 잘 들어가있는 모습!
image

@lydiacho lydiacho added ✨ Feat 새로운 기능 추가 (새로운 구현) labels Oct 30, 2024
@lydiacho lydiacho requested review from ijieun and se0jinYoon October 30, 2024 10:27
@lydiacho lydiacho self-assigned this Oct 30, 2024
Copy link
Collaborator

@se0jinYoon se0jinYoon left a comment

Choose a reason for hiding this comment

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

lgtm!!
저희 근데 폰에 저장되어있는 사진 말고 직접 촬영해서 올렸을 때도 테스트가 되어있나요 ? 아이폰에서 촬영하면 heic 확장자로 파일 전송되는 걸로 아는데 이 확장자도 받을 수 있도록 되어있는지!

는 방금 영상보고왔는데 저희 그냥 파일로만 넣을 수 있게 되어있군요 -> 그렇다면 라이팅이 수정되어야 할 것 같다란 생각이 들긴하네요 . . 이건 좀 더 생각해보고 기디에 말해도 될 듯해요!

확인했숨다~~ 수고하셨어요!!

Copy link
Collaborator

@ijieun ijieun left a comment

Choose a reason for hiding this comment

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

코드 잘봤습니다! 고생하셨습니다!!

@lydiacho lydiacho merged commit e6b0da4 into develop Oct 30, 2024
1 check passed
@lydiacho lydiacho deleted the feat/#316/presignedUrl2 branch October 30, 2024 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feat 새로운 기능 추가 (새로운 구현) size/m
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ Feat ] 명함 사진 Presigned URL 변경
3 participants