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

[POM-94] 가게 이미지 업로드 기능 추가 #38

Merged
merged 12 commits into from
Sep 18, 2023
Merged

Conversation

juno-junho
Copy link
Collaborator

@juno-junho juno-junho commented Sep 17, 2023

📌 설명

  • MultiPartFile의 List를 받아 applicaion-dev.yml 경로에 저장
  • 각 레이어 별 테스트 작성 완료

기능은 MultipartFile로 이미지 리스트를 받으면, 사용자가 전송한 이미지의 이름, 이미지 이름에서 확장자만 떼어 UUID + 확장자이름으로 application-dev.yml에 설정한 경로에 저장합니다.
따라서 StoreImage 엔티티에서는 경로, 원본 이미지 이름, 저장하는 이미지 이름(uuid)로 관리합니다.

참고

지난 pr시 이름 수정, 양방향 매핑, 자잘한 디렉토리 생성 및 application-dev.yml 설정 수정 등으로 file Changed 수가 좀 많습니다.
여러 파일에서 조금씩 수정하였고, 메인은 StoreImage 엔티티와 StoreImageService입니다.
다음부터 좀 더 잘게 PR을 나눠볼게요

@juno-junho juno-junho requested review from hyeon-z, choi5798, beomukim, Juhongseok and ddongpuri and removed request for hyeon-z and choi5798 September 17, 2023 13:31
@juno-junho juno-junho self-assigned this Sep 17, 2023
Copy link
Collaborator

@ddongpuri ddongpuri left a comment

Choose a reason for hiding this comment

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

🍒

Copy link
Collaborator

@hyeon-z hyeon-z left a comment

Choose a reason for hiding this comment

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

👍

@Entity
@EqualsAndHashCode(of = "id")
Copy link
Collaborator

Choose a reason for hiding this comment

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

BaseTimeEntity를 상속하고 있으니 callSuper = false 해당 부분을 추가하면 좋을 것 같습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

callsuper = false는 default라 명시 안했어요

Copy link
Member

@Juhongseok Juhongseok left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Collaborator

@choi5798 choi5798 left a comment

Choose a reason for hiding this comment

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

👍

@juno-junho juno-junho merged commit 570da8c into dev Sep 18, 2023
1 check passed
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.

5 participants