mpcjanssen / simpletask-android

GNU General Public License v3.0
549 stars 128 forks source link

Support for notifications in lua #687

Open jansowinski opened 7 years ago

jansowinski commented 7 years ago

I have no idea how android apps works, but it would be nice and would make simpletask more powerful if you'll implement suport for android notifications in lua.

What I mean by this is just some kind of functions, so that user can config simpletask in lua to display notifications by some given pattern.

For example I could set lua to display all the time sticky notifications for unfinished tasks with threshold date < today.

smichel17 commented 7 years ago

This is totally possible. Right now, though, we don't support notifications at all (we use a calendar to do notifications for us).

I'm planning on adding notifications at some point, but the semester is busy, so we'll see when that happens..

mpcjanssen commented 7 years ago

Crating notifications is not hard, updating and removing them is harder though.

jansowinski commented 7 years ago

Won't some 3-rd party library get this done? Something like this for example?

smichel17 commented 7 years ago

I'm not immediately seeing how that library is that different from just using NotificationCompat.Builder.

That aside, though, the other hard part here is that simpletask doesn't currently keep a good handle on task identity (this being a downside to using todo.txt as a backend). So, for example, the current approach is that simpletask rebuilds the entire calendar each time you change a single task. We'd have to do the same with notifications.

There's #617 for changing to a database backend with plugins to sync the database with a todo.txt file on your phone, dropbox, etc. With that, we have task identity so things like recreating notifications (or rather not recreating notifications when we don't need to) become easier.

jansowinski commented 7 years ago

Ok, so it seems like it doesn't make sense to do notifications without task identity being done first.