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] feat: image 관련 mappedSuperclass로 분리, menu 도메인 엔티티 validation 및 전체적 구조 설정 #43

Merged
merged 8 commits into from
Sep 18, 2023

Conversation

juno-junho
Copy link
Collaborator

📌 설명

  • BaseFileEntity로 파일 관련 중복되는 필드를 모두 분리시켰습니다.
  • 메뉴 등록 관련 전체적인 틀을 생성했습니다
  • 생성한 도메인 엔티티들에 대해서 테스트를 작성했습니다.

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.

👍


private final MenuService menuService;

@PostMapping({"{storeId}"})
Copy link
Collaborator

Choose a reason for hiding this comment

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

중괄호로 한번 더 감싼 이유가 있나요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

중괄호 잘못 들어갔네요 ㅋㅋ

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.

🍫 궁금한 점 하나 남겼어요

private String uploadName;

private String fileName;

@ManyToOne(fetch = LAZY)
@JoinColumn(name = "STORE_ID")
private Store store;

@Builder
private StoreImage(String path, String uploadName, String fileName, Store store) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 private 생성자는 어디에서 호출 되나요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

호출 안되고 builder로만 사용하려고 막아놓은거에요

@juno-junho juno-junho merged commit 8c12a03 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.

4 participants