shacker / django-todo

A multi-user, multi-group todo/ticketing system for Django projects. Includes CSV import and integrated mail tracking.
http://django-todo.org
BSD 3-Clause "New" or "Revised" License
819 stars 285 forks source link

Confirm before delete #97

Closed dalinxz closed 3 years ago

dalinxz commented 4 years ago

Hey shacker, incredible work on this lightweight but powerful todo app! Thank you.

I had one suggestion for future improvements, which is to add a confirm before deleting the task. Keep it up, really appreciate the work.

shacker commented 4 years ago

Good suggestion @dalinxz . I've retitled this issue and marked it "Good first issue" for anyone who drives by and wants to contribute the feature. Thanks.

sum35h commented 3 years ago

Hi @dalinxz , I would like to work on this, if the issue is still open.

shacker commented 3 years ago

Sure thing @sum35h - this still needs to be done, and I welcome your pull request any time. Thanks in advance.

aaroosh-07 commented 3 years ago

@shacker can i take up this issue

shacker commented 3 years ago

@sum35h Have you started on this? If not, do you mind of @aaroosh-07 works on it?

1hanzla100 commented 3 years ago

@shacker I want to fix this issue. Assign it to me

shacker commented 3 years ago

@1hanzla100 I don't assign things here - most OSS projects don't. Either communicate with others in this thread, or just be first with a good pull request!

EdgarPrada commented 3 years ago

This improvement is already done. -Shouldn't this be marked as closed? Great project. Thanks

shacker commented 3 years ago

Thanks @EdgarPrada, right you are. Closing. Glad you enjoy the project!