Delete button deletes item from list and firebase after user confirma…

[granolagabrielle]

Description

This code renders a delete button for each list item. Once the user clicks the button, the app prompts them to confirm that they want to delete the item. Once approved, the item is removed from the list and the database.

Related Issue

closes #12

Acceptance Criteria

• The ListItem component renders a button that allows the user to delete an item from their list when clicked
• Clicking the delete button prompts the user to confirm that they really want to delete the item
• The deleteItem function in api/firebase.js has been filled out, and deletes the item from the Firestore database

Type of Changes

enhancement

Updates

Before

After

Testing Steps / QA Criteria

1. Click the delete button next to an item
2. Select "Ok" from the alert to confirm delete for that item
3. Check firebase to ensure the item was deleted
4. Also select "cancel" to ensure the item stays in the list

[github-actions]

Visit the preview URL for this PR (updated for commit 58047e3):

https://tcl-75-smart-shopping-list--pr49-gb-nk-delete-item-vgf3jdm3.web.app

_{(expires Wed, 25 Sep 2024 18:00:24 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 1f1fd53c369e1fa31e15c310aa075b4e8f4f8dde}

[warrenchan13]

I left one comment. Other than that, it looks good!

[warrenchan13]

based on the failed test, i think you need to import items to use it as a prop for the AddItems component

[kweeuhree]

hey! thank you for catching that! i forgot to push the last changes, i think that this was breaking our deployed app, so i added this change

[deeheber]

This looks good to me and passes all of the acceptance criteria. Tested locally and on the deployed versions.

Going to let this PR sit for a little to see if another mentor also wants to review to get you extra feedback. Otherwise, I'll merge this end of day tomorrow (Sept 19). Let me know if you don't want that to happen.