search

rootkiwi / an2linuxserver

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

Handle error if no notification server is installed #20

Closed podestplatz closed 7 years ago

podestplatz commented 7 years ago

If no notification server is installed self.notif.show() will fail with a g-dbus error

rootkiwi commented 7 years ago

Thanks!

Though you forgot a few ' in the strings which will result in a print with a bunch of whitespaces. You need to have ' at the end of the line and also in beginning of next.

Like this:

print_with_timestamp('Please make sure you have a notification '
        'server installed on your system')

With this:

print_with_timestamp('Please make sure you have a notification \
        server installed on your system')

the print will look like this:

[2017-01-14 20:44:03] Please make sure you have a notification         server installed on your system

Also the backslashes and the semicolon is also not needed.

Please fix that and I will merge.

podestplatz commented 7 years ago

I didn't think of that. Thanks for pointing that out.

Changes should be up.

rootkiwi commented 7 years ago

great!