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

[ARV-175] 차량 대절 사진 저장 로직 및 날짜, 참석자 수 데이터 조회 이슈 수정 #154

Merged
merged 2 commits into from
Jan 1, 2025

Conversation

sangcci
Copy link
Collaborator

@sangcci sangcci commented Jan 1, 2025

연결된 issue 🌟

구현 내용 📢

차량 대절 사진 저장 로직 구현

기존에 차량 대절 추가 및 수정 작업에서 임시로 imageUrl을 바로 받았는데, 이를 multipart form data로 받아와서 S3에 저장할 수 있도록 수정했습니다.

rentboardingDates 조회 쿼리 수정

기존 차량 대절 상세 조회에서 rentBoardingDates 데이터가 1개 밖에 나오지 않는 이슈가 발생하여,
기존에 Projections 안에 list로 한번 더 Projections 했던 것을, 쿼리를 아예 분리하여 DTO에 추가적으로 Set 하도록 변경했습니다.

앞으로 해야할 것

  • s3 transactional 고려 or 비동기 구현
  • 쿼리 시간 측정

- Multipart form data로 JSON, Binary 각각 받아오도록 수정
- Facade 적용
- boardingDates 조회 쿼리 분리
- rentJoin 테이블 일반 join 대신 leftJoin 적용
- 참여자 수가 null이라면 0으로 반환하도록 수정
@sangcci sangcci added the 🐛 fix 수정 작업 label Jan 1, 2025
@sangcci sangcci self-assigned this Jan 1, 2025
@sangcci sangcci merged commit 8f21675 into develop Jan 1, 2025
2 checks passed
@sangcci sangcci deleted the ARV-175-fix/rent branch January 1, 2025 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 fix 수정 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ARV-175] 차량 대절 사진 저장 로직 및 날짜, 참석자 수 데이터 조회 이슈
1 participant