<table><tr><td style="">di72nn added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D22140">View Revision</a></tr></table><br /><div><div><p>I'm not familiar with <tt style="background: #ebebeb; font-size: 13px;">arcanist</tt>, so I'll just add some extra info in a message.</p>

<p>There was a warning message in logs (when a message received in Conversations, for example):</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">org.kde.kdeconnect_tp W/Bundle: Key text expected String but value was a android.text.SpannableString.  The default value <null> was returned.
org.kde.kdeconnect_tp W/Bundle: Attempt to cast generated internal exception:
    java.lang.ClassCastException: android.text.SpannableString cannot be cast to java.lang.String
        at android.os.BaseBundle.getString(BaseBundle.java:1121)
        at org.kde.kdeconnect.Plugins.NotificationsPlugin.NotificationsPlugin.extractConversation(NotificationsPlugin.java:380)
        at org.kde.kdeconnect.Plugins.NotificationsPlugin.NotificationsPlugin.sendNotification(NotificationsPlugin.java:254)
        at org.kde.kdeconnect.Plugins.NotificationsPlugin.NotificationsPlugin.onNotificationPosted(NotificationsPlugin.java:179)
        at org.kde.kdeconnect.Plugins.NotificationsPlugin.NotificationReceiver.onNotificationPosted(NotificationReceiver.java:63)
        at android.service.notification.NotificationListenerService.onNotificationPosted(NotificationListenerService.java:317)
        at android.service.notification.NotificationListenerService$MyHandler.handleMessage(NotificationListenerService.java:1830)
        at android.os.Handler.dispatchMessage(Handler.java:106)
        at android.os.Looper.loop(Looper.java:164)
        at android.app.ActivityThread.main(ActivityThread.java:6494)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:440)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807)</pre></div>

<p>Also <tt style="background: #ebebeb; font-size: 13px;">Notification.EXTRA_IS_GROUP_CONVERSATION</tt> was replaced with a <tt style="background: #ebebeb; font-size: 13px;">NotificationCompat</tt> equivalent which does make a difference on Android <9 (tested on Android 8.1 with Conversations).</p>

<p>Other <tt style="background: #ebebeb; font-size: 13px;">Build.VERSION.SDK_INT</tt>-related changes are not so important, but can improve something on pre-KITKAT devices.</p>

<p>Oh, and thanks for all the great work you guys put into this app!</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R225 KDE Connect - Android application</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22140">https://phabricator.kde.org/D22140</a></div></div><br /><div><strong>To: </strong>di72nn<br /><strong>Cc: </strong>kdeconnect, ewentzel, dshelley, pawelkwiecinski, ctakano, vporvaznik, mschroeder, varunp, shivanshukantprasad, skymoore, fbampaloukas, brute4s99, wistak, anoopv, dvalencia, rmenezes, julioc, Leptopoda, timothyc, Danial0_0, johnq, Pitel, domson, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, lemuel, menasshock, mikesomov, tctara, apol<br /></div>