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

[LIME-58] 리뷰 이미지 추가 #25

Merged
merged 7 commits into from
Jan 25, 2024
Merged

[LIME-58] 리뷰 이미지 추가 #25

merged 7 commits into from
Jan 25, 2024

Conversation

Curry4182
Copy link
Contributor

@Curry4182 Curry4182 commented Jan 25, 2024 •
edited
Loading

📌 PR 종류

어떤 종류의 PR인지 아래 항목 중에 체크 해주세요.

  • 🐛 버그 수정
  • ✨ 기능 추가
  • ✅ 테스트 추가
  • 🎨 코드 스타일 변경 (formatting, local variables)
  • 🔨 리팩토링 (기능 변경 X)
  • 💚 빌드 관련 수정
  • 📝 문서 내용 수정
  • 그 외, 어떤 종류인지 기입 바람:

📌 어떤 기능이 추가 되었나요?

리뷰 이미지 관련 기능 추가 ❤❤

  • 리뷰 등록할 때 이미지 여러개 넣을 수 있도록 기능 추가 92665c7
  • 리뷰 조회할 때 이미지 조회할 수 있도록 기능 추가 8650439

Issue Number

LIME-58

기능 설명

리뷰 사진 저장 ❤❤

  • 프론트에서 form-data로 Multipartfile을 전송 받으면 서버에서 네이버 클라우드 버킷으로 전송합니다.
  • 이 후 네이버 클라우드에 저장되어있는 이미지의 주소를 리뷰 이미지 테이블에 리뷰 아이디를 pk로 하여 저장합니다. 632131e 92665c7

리뷰 목록 조회 ❤❤

📌 기존에 있던 기능에 영향을 주나요?

  • 네
  • 아니요

@Curry4182 Curry4182 added the enhancement New feature or request label Jan 25, 2024
@Curry4182 Curry4182 self-assigned this Jan 25, 2024
HandmadeCloud
HandmadeCloud approved these changes Jan 25, 2024
View reviewed changes
Copy link
Member

@HandmadeCloud HandmadeCloud left a comment

Choose a reason for hiding this comment

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

수고하셨습니다ㅏ. 직접 프론트까지 구현해서 확인하는 습관은 배우고 싶네용

Yiseull
Yiseull approved these changes Jan 25, 2024
View reviewed changes
Copy link
Member

@Yiseull Yiseull left a comment

Choose a reason for hiding this comment

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

병곤님 고생 많으셨습니당!

lime-domain/src/main/java/com/programmers/lime/domains/review/domain/ReviewImage.java Outdated
Comment on lines 22 to 23
@Column(name = "review_id", nullable = false)
private Long reviewId;
Copy link
Member

Choose a reason for hiding this comment

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

p1;
리뷰 이미지는 리뷰가 삭제될 때, 같이 삭제되어야 하는데 객체 참조를 안해준 이유가 있나용?? 그때 팀에서 정한 바로는 같이 삭제되는 관계에 대해서는 객체 참조를 하기로 정한 것으로 기억합니다!

Copy link
Contributor Author

@Curry4182 Curry4182 Jan 25, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

JoinColumn 설정해서 review가 삭제되면 review image도 삭제 되도록 변경했습니다~! 31c6a64

@Curry4182 Curry4182 merged commit ffb7968 into main Jan 25, 2024
@Curry4182 Curry4182 deleted the LIME-58--review-image-feat branch February 7, 2024 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Yiseull Yiseull Yiseull approved these changes

@HandmadeCloud HandmadeCloud HandmadeCloud approved these changes

Assignees

@Curry4182 Curry4182

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Curry4182 @HandmadeCloud @Yiseull