rootkiwi / an2linuxserver

Sync Android notifications encrypted to a Linux desktop
Other
401 stars 43 forks source link

Allow notifications to be ignored if contain specific keywords #38

Closed ammgws closed 5 years ago

ammgws commented 5 years ago

Added ability to ignore notifications (prevent from showing) if they contain specific keywords set in the config file.

Useful if you have many devices and would rather configure it once server side rather than fiddling with the notification settings in the app on each device.

Might have it search the notificaiton message text too.

rootkiwi commented 5 years ago

Great thank you, this can be useful! You say WIP, you want to add something before I merge this? Like the message text? I'm ok either way.

ammgws commented 5 years ago

I thought I would leave it open for comments and if there are any other things we could add them in before merging.

For example I saw that appname isn't sent by the client to the server, but that could be pretty useful server side in terms of logging and ignoring etc. However I haven't had time to play with the client app code yet.

rootkiwi commented 5 years ago

Okey yea sure.

Yes the appname is not sent, but that could be useful as you say. Also support for logging on server side would be nice.

ammgws commented 5 years ago

Should we merge this for now and leave the other features to another PR?

(I have been using this for a week or so now)

rootkiwi commented 5 years ago

Yes, merging now. Thank you!