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: 기록 생성 시 Image의 FK를 채우는 기능 구현, 기록에 Pool 저장 구현 #46

Merged
merged 5 commits into from
Jul 16, 2024

Conversation

ywonchae1
Copy link
Member

@ywonchae1 ywonchae1 commented Jul 15, 2024

🌱 관련 이슈

📌 작업 내용 및 특이사항

기록 생성 플로우 정리

  1. 이미지부터 업로드 API를 호출합니다. 이때 이미지는 기록과 연결되지 않고 FK가 Null로 존재합니다.
  2. 응답으로 이미지 ID List 얻습니다.
  3. 기록 내용을 이미지 ID List와 함께 담아 기록 저장 API를 호출합니다.
  4. 기록 저장 기능에서는 기록을 저장하고, 저장된 기록의 ID값을 Null로 존재하던 이미지의 FK에 업데이트하여 연결합니다.

구현 사항
image

  • 기존에는 위와 같이 업로드된 이미지에 기록 ID를 추가하는 로직(addMemoryToImages)이 분리되어 있었습니다.
  • 별도로, 수영 기록의 이미지를 수정하는 로직(updateImages)과 EndPoint가 겹치는 문제가 있었습니다.
  • 이미지에 기록 ID를 추가하는 로직은 기록을 저장하는 로직과 연결되어 있어야 하는 것으로 판단, 기록 저장 로직에 추가하였습니다.

Pool 연결

  • 요청 body로 받은 Pool Id에 따라 수영장 정보를 찾고, 기록에 반영합니다.

📝 참고사항

  • MemoryControllerImageUploadService에 의존하고 있습니다.
  • 먼저 memoryService.save()를 호출하고, 결과가 Null이면 ApiResponse.fail을 응답합니다.
  • 기록이 잘 저장되어 Null이 아니라면, imageUploadService.addMemoryIdToImages()를 호출하여 이미지에 기록 ID를 추가합니다.
  • 이때 이미지는 한 개일 수도 있고, 여러 개일 수도 있습니다.
  • 성공하면 ApiResponse.success를 응답합니다.

@ywonchae1 ywonchae1 linked an issue Jul 15, 2024 that may be closed by this pull request
@penrose15
Copy link
Collaborator

수고하셨습니다. 코드 깔끔하시네요 👍 컨플릭트 해결하고 머지하면 될 것 같습니다.

@ywonchae1 ywonchae1 merged commit 07f3fb6 into develop Jul 16, 2024
@ywonchae1 ywonchae1 deleted the feature/45-memory-create-image-update branch July 16, 2024 01:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ 기록 저장 시 이미지 fk 수정
2 participants