D8966: Add a foreground notification to the background service

[Matthijs Tijink]

mtijink added a comment.


  I think `startForegroundService` is only a promise that you'll call `startForeground` from the service. So `startForeground` is needed in any case. Without it, it might not even start the service (because of background limitations).
  
  Regarding the support library problem: I think you can decide to call `startService` or `startForegroundService` manually, depending on the API version, even without the newer support library, so it should work on all android versions. I cannot actually test this as I don't have an Android Oreo device.

REPOSITORY
  R225 KDE Connect - Android application

REVISION DETAIL
  https://phabricator.kde.org/D8966

To: mtijink, #kde_connect
Cc: albertvaka, nicolasfella
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20171209/597bd588/attachment-0001.html>