Add Android notification channel support
ClosedPublic
Actions

Authored by vkrause on Jan 16 2019, 7:13 PM.

Details

Reviewers

Commits

R289:1c3c1dd70980: Add Android notification channel support

Summary

This is required for API level 28 or higher. This actually matches the
notfication events KNotification defines quite nicely, so those can be
individually configured now, like we are used to on other platforms.

Diff Detail

Repository

R289 KNotifications

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

vkrause created this revision.Jan 16 2019, 7:13 PM

Restricted Application added a project: Frameworks. · View Herald TranscriptJan 16 2019, 7:13 PM

Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript

vkrause requested review of this revision.Jan 16 2019, 7:13 PM

Harbormaster completed remote builds in B7193: Diff 49661.Jan 16 2019, 7:13 PM

LGTM.

This revision is now accepted and ready to land.Jan 19 2019, 5:56 AM

Closed by commit R289:1c3c1dd70980: Add Android notification channel support (authored by vkrause). · Explain WhyJan 19 2019, 10:29 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			src/android/org/kde/knotifications/KNotification.java (3 lines)
M			src/android/org/kde/knotifications/NotifyByAndroid.java (22 lines)
M			src/notifybyandroid.h (2 lines)
M			src/notifybyandroid.cpp (14 lines)