Bug 408527

Summary: Phone led keeps blinking after answering a text with KDE Connect
Product: [Applications] kdeconnect Reporter: Raphaël Jakse <raphael.kde>
Component: android-applicationAssignee: Albert Vaca Cintora <albertvaka>
Status: RESOLVED WORKSFORME    
Severity: normal    
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Raphaël Jakse 2019-06-10 13:52:16 UTC
When I answer a text with the computer from KDE Connect, the phone's led notification keeps blinking. Though it is obvious that I read the very message that makes the phone blink in the first place. And I don't really want to unlock the phone to make it stop blinking.

If it's not just me, I suspect it's not easy to fix (it seems to involve controlling the notification of the SMS app? Maybe it requires some kind of integration with this app?), but I did not find any bug report on this so here it is.
Comment 1 Albert Vaca Cintora 2019-06-10 14:45:37 UTC
The notification gets dismissed but the LED stays? In that case I
don't think we can do much. If the notification doesn't get dismissed,
then I guess we just could dismiss it after replying.

On Mon, Jun 10, 2019 at 3:52 PM Raphaël Jakse <bugzilla_noreply@kde.org> wrote:
>
> https://bugs.kde.org/show_bug.cgi?id=408527
>
>             Bug ID: 408527
>            Summary: Phone led keeps blinking after answering a text with
>                     KDE Connect
>            Product: kdeconnect
>            Version: unspecified
>           Platform: Other
>                 OS: Linux
>             Status: REPORTED
>           Severity: normal
>           Priority: NOR
>          Component: android-application
>           Assignee: albertvaka@gmail.com
>           Reporter: raphael.kde@jakse.fr
>   Target Milestone: ---
>
> When I answer a text with the computer from KDE Connect, the phone's led
> notification keeps blinking. Though it is obvious that I read the very message
> that makes the phone blink in the first place. And I don't really want to
> unlock the phone to make it stop blinking.
>
> If it's not just me, I suspect it's not easy to fix (it seems to involve
> controlling the notification of the SMS app? Maybe it requires some kind of
> integration with this app?), but I did not find any bug report on this so here
> it is.
>
> --
> You are receiving this mail because:
> You are the assignee for the bug.
Comment 2 Raphaël Jakse 2019-06-10 14:48:56 UTC
> The notification gets dismissed but the LED stays?

No, the notification (produced by QKSMS, but I think I already had the issue with the regular SMS app - I cannot reproduce, because since I updated by phone to Android Pie, the Led does not blink anymore with the regular SMS app) is still there.

Is it possible for KDE Connect to dismiss a notification from QKSMS?
Comment 3 Albert Vaca Cintora 2019-06-10 14:50:28 UTC
KDE Connect can dismiss notifications, so it should be possible.

Although... I would say they should dismiss it themselves after replying actually :/ maybe we can consider this a bug on their side?
Comment 4 Raphaël Jakse 2019-06-10 14:58:17 UTC
Sure.

Relevant bug: https://github.com/moezbhatti/qksms/issues/1445
Comment 5 Raphaël Jakse 2019-06-10 15:08:32 UTC
Would it be desirable to implement a "notification dismiss workaround" in KDE Connect that one could enable?

Or, the notification could be dismissed as soon as the message is displayed on the computer, which QKSMS can't know.
Comment 6 Raphaël Jakse 2019-06-10 15:13:02 UTC
Well, I just sent a reply to a text from KDE connect and the led stopped blinking. Weird.

However my last suggestion could still be worth it, so I let the bug open so you can decide what is best.
Comment 7 Raphaël Jakse 2019-06-12 09:03:45 UTC
As a workaround, I just found that manually dismissing the notification from KDE Connect is possible and works well. Which is not perfect but convenient enough.
Comment 8 Raphaël Jakse 2019-09-21 12:44:30 UTC
I can't reproduce this bug anymore.