huytd / pomoday-v2

A keyboard only task management web app
https://pomoday.vercel.app
BSD 3-Clause "New" or "Revised" License
586 stars 69 forks source link

Enabled archive task by done status #61

Closed nhvu1988 closed 4 years ago

nhvu1988 commented 4 years ago

User can use current archive command with #done to archive all done tasks

huytd commented 4 years ago

Hey, thanks for the PR. However, I would like to give some feedback and request.

Since you're making this change, please make it work for all other task status as well, just like the show/hide command, which also works for task status, ids,...

Also, in show/hide command, we use task status as is, without the #, so please make archive command behave the same way.

Thank you so much!

nhvu1988 commented 4 years ago

I'm sorry but I don't follow this PR anymore, thanks for your advice :)