SauveJeanLuc / coffee-tasks

Next gen AI Powered Todo-List
https://coffeetasks.netlify.app
MIT License
40 stars 73 forks source link

Delete a task should have a confirmation dialog #134

Closed manassahoo-dev closed 1 year ago

manassahoo-dev commented 1 year ago

Prerequisites

Please answer the following questions for yourself before submitting an issue.

Describe the bug \ Delete a task should have a confirmation dialog

To Reproduce \ Steps to reproduce the behavior:

  1. Click on Delete icon on any of the task
  2. Task gets deleted

Expected behavior \ Delete Click event should open a confirmation message. Based on that user can delete the task. This will help preventing the mistake to delete a task.

Current behaviour \ A clear and consise description of what is happening. If possible, give reasoning behind difference in current behaviour and expected behaviour

Possible solutions (Optional) \ Give ideas for solving the problem, provide relevant links to additional packages you are mentioning

Screenshots

Desktop (please complete the following information):

Failure Logs

Please include any relevant log snippets or files here.

Additional context Add any other context about the problem here.

License

manassahoo-dev commented 1 year ago

Screencast from 15-10-22 12:12:57 PM IST.webm

SauveJeanLuc commented 1 year ago

You are right, Go for it @manassahoo-dev

manassahoo-dev commented 1 year ago

@SauveJeanLuc I am getting the error while running in my local machine. image

uKiJo commented 1 year ago

I added the feature, take a look at the pull request!

SauveJeanLuc commented 1 year ago

@SauveJeanLuc I am getting the error while running in my local machine. image

remove node_modules and redo npm i