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

[feat] 음식 상품 수량 등록 #76

Merged
merged 10 commits into from
Aug 16, 2024

Conversation

june-777
Copy link
Member

💡 다음 이슈를 해결했어요.

Issue Link - #59

  • 비즈니스 요구사항을 최소화하기 위해, 음식 상품 수량은 점주가 음식 상품을 등록할 때 함께 등록합니다.
  • 따라서 별도의 API는 생성하지 않았으며, 음식 상품(Menu)의 인스턴스 필드와 생성자에 재고수(stockCount)를 추가하였습니다.

✅ 셀프 체크리스트

  • 내 코드를 스스로 검토했습니다.
  • 필요한 테스트를 추가했습니다.
  • 모든 테스트를 통과합니다.
  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있습니다.
  • 커밋 메세지를 컨벤션에 맞추었습니다.
  • wiki를 수정했습니다.

@june-777 june-777 self-assigned this Aug 15, 2024
@june-777 june-777 added the ✨ Feature 기능 개발 label Aug 15, 2024
@june-777 june-777 added this to the 프로토타입 만들기 milestone Aug 15, 2024
Copy link
Member

@Dr-KoKo Dr-KoKo left a comment

Choose a reason for hiding this comment

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

1)메뉴를 등록할 때 재고 개수를 같이 입력한다.
2-1) 재고가 남아있으면 주문이 성공한다.
2-2) 재고가 소진되면 주문이 실패한다.
의 동작을 생각하고 계신걸까요?

점주에게 알림을 띄우지 않아도 되기 때문에 프로젝트 사이즈가 확실히 줄겠다고 생각하는데 다른 분들 의견도 한번 들어보는게 좋을 것 같아요.
@Hyeon-Uk @kimhyun5u

@june-777
Copy link
Member Author

june-777 commented Aug 15, 2024

@Dr-KoKo
점주가 음식 상품 등록할 때, 수량도 함께 입력하여 음식 상품을 등록하도록 한 것입니다. (Create, Update 중 Create만)
기존의 요구사항에서 크게 변동되는 부분은 없는 것 같은데, 아래의 이야기는 어떤 말씀이실까요??

점주에게 알림을 띄우지 않아도 되기 때문에

@Dr-KoKo
Copy link
Member

Dr-KoKo commented Aug 15, 2024

@Dr-KoKo 점주가 음식 상품 등록할 때, 수량도 함께 입력하여 음식 상품을 등록하도록 한 것입니다. (Create, Update 중 Create만) 기존의 요구사항에서 크게 변동되는 부분은 없는 것 같은데, 아래의 이야기는 어떤 말씀이실까요??

점주에게 알림을 띄우지 않아도 되기 때문에

재고를 추가로 등록하는게 아니라 Menu 생성시점에 재고를 바로 입력받자는 말씀이신가보네요.

"비즈니스 요구사항을 최소화하기 위해, 음식 상품 수량은 점주가 음식 상품을 등록할 때 함께 등록합니다."

저는 점주가 재고를 갱신하는 기능을 생각했는데 "점주가 음식 상품을 등록할 때 함께 등록합니다"로 결정하시게 된 배경이 궁금합니다. 혹시 구현하시다보니 사이즈가 생각보다 크던가요?

@june-777
Copy link
Member Author

네네 맞습니다.

첫 번째 이유는 PR 단위를 짧게 가져가기 위함이였고,
두 번쨰 이유는 현재 정의된 백로그들이 아직 많은 상황이라 CRUD에서 U,D는 우선순위가 낮다고 판단했습니다.
따라서 추후에 필요시에 구현해도 될 것이라 생각했습니다 :)

@Dr-KoKo
Copy link
Member

Dr-KoKo commented Aug 15, 2024

네네 맞습니다.

첫 번째 이유는 PR 단위를 짧게 가져가기 위함이였고, 두 번쨰 이유는 현재 정의된 백로그들이 아직 많은 상황이라 CRUD에서 U,D는 우선순위가 낮다고 판단했습니다. 따라서 추후에 필요시에 구현해도 될 것이라 생각했습니다 :)

아하 알겠습니다. 금일 10시에 회의하면서 어디까지 쳐낼지 논의해보면 좋을 것 같네요. 알림기능을 완전히 쳐내는 것도 한 방안일 것 같아서 말씀드려봤습니다!

@june-777
Copy link
Member Author

아하 알겠습니다. 금일 10시에 회의하면서 어디까지 쳐낼지 논의해보면 좋을 것 같네요. 알림기능을 완전히 쳐내는 것도 한 방안일 것 같아서 말씀드려봤습니다!

좋습니다!

Copy link
Member

@Dr-KoKo Dr-KoKo left a comment

Choose a reason for hiding this comment

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

금일 논의를 거쳐 (점주의 승락/거절 절차 없이) 재고가 있으면 자동승락, 없으면 자동거절되게 요구사항이 변경되었습니다.

…식상품수량등록

# Conflicts:
#	src/test/java/camp/woowak/lab/cart/service/CartServiceTest.java
#	src/test/java/camp/woowak/lab/web/api/cart/CartApiControllerTest.java
Copy link
Contributor

@Hyeon-Uk Hyeon-Uk left a comment

Choose a reason for hiding this comment

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

테스트 빨리 고치시길 기원하겠습니다 :-) 🙏

@june-777
Copy link
Member Author

테스트 빨리 고치시길 기원하겠습니다 :-) 🙏

754e122 fix 완료했습니다 🙏

@june-777 june-777 merged commit 3770fa0 into main Aug 16, 2024
1 check passed
@june-777 june-777 deleted the feature/59_june-777_음식상품수량등록 branch August 16, 2024 02:07
@june-777 june-777 linked an issue Aug 16, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[기능] 점주의 음식 상품 수량 등록
3 participants