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

Compare shopping list to pantry #35

Closed
9 tasks done
blahosyl opened this issue Apr 16, 2024 · 1 comment · Fixed by #43
Closed
9 tasks done

Compare shopping list to pantry #35

blahosyl opened this issue Apr 16, 2024 · 1 comment · Fixed by #43
Assignees
Labels
enhancement New feature or request optional Outside the scope of the MVP

Comments

@blahosyl
Copy link
Owner

blahosyl commented Apr 16, 2024

User goal: adjust the shopping list based on the ingredients already available

Possible implementation:

  • At the end of the program, before the goodbye screen, ask user if they want to compare shopping list to pantry
    • validate answer (Y/N)
  • loop through each ingredient, and ask how much they have
    • can only enter a float (can use a similar validation as for dish selection)
    • subtract entered number from quantity on shopping list
    • if shopping list quantity <= zero, delete item from shopping list
  • print revised shopping list
  • make new function a method of the ShoppingList object
  • integrate new function into existing flow
@blahosyl blahosyl added enhancement New feature or request optional Outside the scope of the MVP labels Apr 16, 2024
@blahosyl blahosyl self-assigned this Apr 16, 2024
blahosyl added a commit that referenced this issue Apr 16, 2024
blahosyl added a commit that referenced this issue Apr 16, 2024
after checking with the pantry, only items with a quantity of more than 0 should remain on the shopping list
blahosyl added a commit that referenced this issue Apr 16, 2024
blahosyl added a commit that referenced this issue Apr 16, 2024
blahosyl added a commit that referenced this issue Apr 17, 2024
@blahosyl blahosyl linked a pull request Apr 17, 2024 that will close this issue
blahosyl added a commit that referenced this issue Apr 17, 2024
move outside the scope of `ShoppingList`
blahosyl added a commit that referenced this issue Apr 17, 2024
blahosyl added a commit that referenced this issue Apr 18, 2024
blahosyl added a commit that referenced this issue Apr 18, 2024
blahosyl added a commit that referenced this issue Apr 18, 2024
blahosyl added a commit that referenced this issue Apr 18, 2024
@blahosyl blahosyl reopened this Apr 18, 2024
@blahosyl
Copy link
Owner Author

Add logic for case where all ingredients are in the pantry, so shopping list becomes empty

blahosyl added a commit that referenced this issue Apr 18, 2024
blahosyl added a commit that referenced this issue Apr 18, 2024
this happens when all ingredients are in the pantry in sufficient quentities
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request optional Outside the scope of the MVP
Projects
Development

Successfully merging a pull request may close this issue.

1 participant