Bug 367930 - SMS text message and Notifications conflict.
Summary: SMS text message and Notifications conflict.
Status: RESOLVED FIXED
Alias: None
Product: kdeconnect
Classification: Applications
Component: common (show other bugs)
Version: 1.4
Platform: Compiled Sources Linux
: NOR major
Target Milestone: ---
Assignee: Albert Vaca Cintora
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-08-28 17:41 UTC by rburns
Modified: 2018-10-10 15:45 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description rburns 2016-08-28 17:41:29 UTC
On the desktop when a text message is received, a helpful notification will appear asking whether or not to reply with a text message. This notification will only last a fraction of a second before it is replaced with the old generic notification that will pop up.  And it will almost always result in losing connectivity with the phone requiring the user to manually pair the devices again.  Using git version compiled Aug 27.

Reproducible: Always

Steps to Reproduce:
1. Pair devices
2. Send sms text message to phone
3. Watch bottom right hand corner for notifications.

Actual Results:  
Option to reply will not be accessible as it disappears, being replaced with a generic notification.

Expected Results:  
User to be able to click the 'Reply' button on the notification.

I did manage to snag the reply button once, and it opened up a text box to reply with.  There does not appear to be any other way to access this functionality, so if you miss the button you're kind of screwed.  Git version was used as the arch package is taking it's sweet time to be updated.
Comment 1 ttv200 2016-08-31 21:25:59 UTC
It's happens to me too.
I thought its because I use custom SMS app, but the problem exist also in the stock sms app.
I use CyanogenMod 6.0.1.
Comment 2 Nick Cross 2016-09-19 08:43:25 UTC
I can confirm I can see something like this as well. Note I do *not* loose pairing with the phone but I can replicate the popup issue.

Now but 368172 is fixed I was trying out the new features and sending a test message to myself I see the popup, which, if I'm *really* quick I can click the button on and then reply (which works fine!). 

I think there needs to be a better way of accessing this feature.
Comment 3 Albert Vaca Cintora 2016-09-20 10:47:09 UTC
Git commit 3b5a7deebee3cd5c7b98fe2f5271918cba854fff by Albert Vaca.
Committed on 20/09/2016 at 10:46.
Pushed by albertvaka into branch '1.x'.

Rewritten the Notification Filter.
- Now we display every app in the phone.
- We show the app icon next to the name.
- Some apps can now be disabled by default.

Disabled notifications for the default Android messaging app, as SMS
notifications are already handled by the telephony plugin.

M  +8    -0    res/layout/activity_notification_filter.xml
M  +7    -0    src/org/kde/kdeconnect/Helpers/StringsHelper.java
M  +35   -45   src/org/kde/kdeconnect/Plugins/NotificationsPlugin/AppDatabase.java
M  +115  -98   src/org/kde/kdeconnect/Plugins/NotificationsPlugin/NotificationFilterActivity.java
M  +0    -5    src/org/kde/kdeconnect/Plugins/NotificationsPlugin/NotificationsPlugin.java

http://commits.kde.org/kdeconnect-android/3b5a7deebee3cd5c7b98fe2f5271918cba854fff
Comment 4 Albert Vaca Cintora 2016-09-20 10:55:05 UTC
The app in the Beta channel of the Play Store now has the default SMS app notifications disabled, so they don't appear at the same time than the SMS plugin notifications. Can you test if this works for you?

If you use a different SMS app, it would be nice if you can test if disabling its notifications from the notifications plugin config fixes the problem. If it does, please tell me which SMS app you use and I will add it to the list of ignored notifications.
Comment 5 Nick Cross 2016-09-20 11:19:50 UTC
I use the default app. The problem for me is not on the android side but in the desktop side. The notification on the desktop doesn't last for sufficient time and there is no way of getting it back.