Skip to content

KNU-1 feat: 주막 Booth & 주막 Menu 도메인 모델(엔티티) 설계#4

Merged
GyunHeee merged 6 commits intomainfrom
feature/knu-1
Feb 9, 2026
Merged

KNU-1 feat: 주막 Booth & 주막 Menu 도메인 모델(엔티티) 설계#4
GyunHeee merged 6 commits intomainfrom
feature/knu-1

Conversation

@GyunHeee
Copy link
Contributor

@GyunHeee GyunHeee commented Feb 3, 2026

✨ 구현한 기능

  • 주막 Booth 도메인 모델(엔티티) 설계
  • 주막 Menu 도메인 모델(엔티티) 설계

📢 논의하고 싶은 내용

🎸 기타

@GyunHeee GyunHeee self-assigned this Feb 5, 2026
Copy link
Contributor

@wlgns12370 wlgns12370 left a comment

Choose a reason for hiding this comment

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

디스커션으로 이야기한대로 소프트 딜리트가 필요한지 이야기해보고 필요하다면 반영 부탁드립니다.

Comment on lines 42 to 49
public static PubMenu of(String name, int price, String imageUrl, PubBooth pubBooth) {
return PubMenu.builder()
.name(name)
.price(price)
.imageUrl(imageUrl)
.pubBooth(pubBooth)
.build();
}
Copy link
Contributor

@wlgns12370 wlgns12370 Feb 6, 2026

Choose a reason for hiding this comment

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

of 메서드 명을 createPubMenu로 변경해야할 것 같습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

네 수정했습니다

Copy link
Contributor

@milk-stone milk-stone left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@GyunHeee GyunHeee merged commit 6d3c265 into main Feb 9, 2026
1 check passed
@GyunHeee GyunHeee deleted the feature/knu-1 branch February 9, 2026 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants