Bug 474643 - Cannot enable notification permissions on Android 13.
Summary: Cannot enable notification permissions on Android 13.
Status: RESOLVED FIXED
Alias: None
Product: KDE Itinerary
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Android Android 13.x
: NOR normal
Target Milestone: ---
Assignee: Volker Krause
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-09-18 04:46 UTC by idoitprone
Modified: 2023-10-12 15:06 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Per app notification enable toggle screenshot. (65.83 KB, image/png)
2023-09-18 04:46 UTC, idoitprone
Details
Screen shot of the commit working (84.57 KB, image/png)
2023-09-21 19:37 UTC, idoitprone
Details

Note You need to log in before you can comment on or make changes to this bug.
Description idoitprone 2023-09-18 04:46:47 UTC
Created attachment 161688 [details]
Per app notification enable toggle screenshot.

SUMMARY
***
NOTE: If you are reporting a crash, please try to attach a backtrace with debug symbols.
See https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports
***


STEPS TO REPRODUCE
1. Click Settings
2. Click Configure Notification


OBSERVED RESULT
The phone believes "This app does not send notification". I cannot enable notifications

EXPECTED RESULT
I can enable notifications

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma:  
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 5.109.0 
Qt Version: 5.15.10

Android: 13

ADDITIONAL INFORMATION
Comment 1 idoitprone 2023-09-18 04:47:55 UTC
23.11.70 (1694968095) f-droid
Comment 2 Bug Janitor Service 2023-09-19 14:53:25 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/knotifications/-/merge_requests/134
Comment 3 Volker Krause 2023-09-19 14:54:44 UTC
Git commit 843d76319f17dd6c315aa2f3efbb428c23f16908 by Volker Krause.
Committed on 19/09/2023 at 16:54.
Pushed by vkrause into branch 'release/23.08'.

Require POST_NOTIFICATION permission for Android SDK 33

M  +1    -0    src/app/android/AndroidManifest.xml

https://invent.kde.org/pim/itinerary/-/commit/843d76319f17dd6c315aa2f3efbb428c23f16908
Comment 4 idoitprone 2023-09-21 19:37:08 UTC
Created attachment 161787 [details]
Screen shot of the commit working

Fdroid 23.11.70 (1695281007) 
build ameabi-7va
Comment 5 idoitprone 2023-09-21 19:38:07 UTC
Closing bug because the commit fixes the bug, I may need to test whether the notification works later.
Comment 6 Volker Krause 2023-09-22 14:17:37 UTC
Thanks for testing/confirming! This isn't the complete fix yet though, Itinerary should ask for the necessary permissions itself, but that is still being worked on.
Comment 7 Volker Krause 2023-10-11 16:08:03 UTC
Git commit 63f4acc6e1c59563bd6165ee513540390d3adf7f by Volker Krause.
Committed on 11/10/2023 at 18:06.
Pushed by vkrause into branch 'master'.

Adapt to notification API and permission changes in Android SDK 33

This still doesn't make notifications work out of the box again (the
application would need to explicitly ask for runtime permissions),
but at least it's working again when that is done or the user manually
enables notification permissions.

M  +3    -1    src/android/AndroidManifest.xml
M  +20   -5    src/android/org/kde/knotifications/NotifyByAndroid.java

https://invent.kde.org/frameworks/knotifications/-/commit/63f4acc6e1c59563bd6165ee513540390d3adf7f
Comment 8 Bug Janitor Service 2023-10-11 16:19:03 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/knotifications/-/merge_requests/137
Comment 9 Volker Krause 2023-10-12 15:06:25 UTC
Git commit 68f0b5a67c0f8aee9019d889ffae5c032599a832 by Volker Krause.
Committed on 11/10/2023 at 18:18.
Pushed by vkrause into branch 'kf5'.

Adapt to notification API and permission changes in Android SDK 33

This still doesn't make notifications work out of the box again (the
application would need to explicitly ask for runtime permissions),
but at least it's working again when that is done or the user manually
enables notification permissions.


(cherry picked from commit 63f4acc6e1c59563bd6165ee513540390d3adf7f)

M  +3    -1    src/android/AndroidManifest.xml
M  +20   -5    src/android/org/kde/knotifications/NotifyByAndroid.java

https://invent.kde.org/frameworks/knotifications/-/commit/68f0b5a67c0f8aee9019d889ffae5c032599a832