Issue #5 | Add Item to Shopping List

dilekbaykara commented 2 months ago

Description

This PR allows users to add items within a form to the database in order to track the future replenishment date of their items.

After adding an item in the input value, there are three options in the form:

Soon (corresponds to 7 days)
Kind of soon (corresponds to 14 days)
Not soon (corresponds to 30 days)

Each of the above days assign a date to that item starting from the date the item was added to the database.

Related Issue

Closes #5

Acceptance Criteria

[x] The ManageList view displays a form that allows them to enter the name of the item and select how soon they anticipate needing to buy it again. There should be 3 choices for this:
- “Soon”, corresponding to 7 days
- “Kind of soon”, corresponding to 14 days
- “Not soon”, corresponding to 30 days
[x] The input that accepts the name of the item has a semantic label element associated with it
[x] The user can submit this form with both the mouse and the Enter key
[x] When the user submits the form, they see a message indicating that the item either was or was not saved to the database.

Data-related tasks:

[x] The console.log in the addItem function in src/api/firebase.js is replaced with a function that adds the new document to the Firestore database. That function will be imported from the firebase/firestore module.
[x] The user’s soon/not soon/kind of soon choice is used to calculate nextPurchasedDate

Type of Changes

enhancement

Updates

Before

After

Testing Steps / QA Criteria

Sign in
Enter name of Item
Select option for date
Click Submit
Check if item appears in database in firestore page
Test alert if user does not input item name when submit button is clicked
Test alert if user does not select date when submit button is clicked

github-actions[bot] commented 2 months ago

Visit the preview URL for this PR (updated for commit 49a4dcd):

https://tcl-75-smart-shopping-list--pr30-nk-db-add-new-item-t-a5e657vs.web.app

_{(expires Fri, 30 Aug 2024 15:54:42 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 1f1fd53c369e1fa31e15c310aa075b4e8f4f8dde}

kweeuhree commented 2 months ago

These changes look great, and the code is clear and concise. One note is that the user is able to add multiple of the same item. Do we want it to behave that way, or should we have another alert for when an item is duplicated? My preference would be to not allow duplicated items, but I'm open to other opinions!

Great point!

deeheber commented 2 months ago

These changes look great, and the code is clear and concise. One note is that the user is able to add multiple of the same item. Do we want it to behave that way, or should we have another alert for when an item is duplicated? My preference would be to not allow duplicated items, but I'm open to other opinions!

Great observation, though I don't think y'all need to take any action on this now since it will be addressed as part of issue #10 that will be assigned in later weeks.

the-collab-lab / tcl-75-smart-shopping-list