Skip to content

fix: 재료 삭제 시 삭제할 수량 입력#35

Merged
MinFe4990 merged 1 commit intodevelopfrom
hongbeom
Dec 15, 2024
Merged

fix: 재료 삭제 시 삭제할 수량 입력#35
MinFe4990 merged 1 commit intodevelopfrom
hongbeom

Conversation

@KIM9909
Copy link
Copy Markdown
Contributor

@KIM9909 KIM9909 commented Dec 15, 2024

📋 요약

  • 원래는 재료 삭제할 때 수량에 상관 없이 바로 삭제가 되었지만, 재료 삭제 시 수량 입력 로직을 추가함
  • 남아 있던 수량과 삭제 수량이 일치하면 해당 재료 완전 삭제
  • 남아 있던 수량보다 삭제할 수량이 더 적다면 남아있는 수량이 자동 계산된 후 저장 됨

🛠 변경 사항

  • 이번 Pull Request에서 작업한 주요 변경 사항은 다음과 같습니다:
    • 코드 리팩토링 또는 최적화

🔗 관련 이슈

📸 스크린샷 또는 GIF (해당되는 경우)

  • 삭제
    image
    삭제시 deleteAmount를 사용자가 직접 입력

✅ 체크리스트

🛡 테스트 방법

📚 추가 참고 사항

  • 이 PR과 관련해 리뷰어가 알아야 할 추가 내용이 있다면 작성해주세요.

@yunjaeeun
Copy link
Copy Markdown
Contributor

로직 구현이 끝나면 제 기능도 연결되게 수정해야 될 것 같네요

@uni-zun
Copy link
Copy Markdown
Contributor

uni-zun commented Dec 15, 2024

check !!

Copy link
Copy Markdown
Contributor

@LeeGukgeon LeeGukgeon 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
Copy Markdown
Contributor

@kingkang85 kingkang85 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
Copy Markdown
Contributor

@MinFe4990 MinFe4990 left a comment

Choose a reason for hiding this comment

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

!check

@MinFe4990 MinFe4990 merged commit 56fd480 into develop Dec 15, 2024
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.

6 participants