-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New requirement for Android: Target SDK: 26 #39
Comments
One thing that I found out the hard way with 26 is that you now have to call NotificationManager.createNotificationChannel(), ideally during plugin initialization. Otherwise you get either nothing, or a toast telling you that you must create a channel (the latter is a switch in the Android developer setting). I hacked it into my own fork at vbraun@c57c3db. Note that the channel also carries a human-readable name, which probably should be configurable in a polished solution. |
Thanks vbraun. I understand you have been succesfully compiling and running Android app with higher cordova and platform versions.
|
I'm using it with cordova 6.5.0, cordova-android 6.4.0, no ios. I haven't updated cordova yet since I'm using another plugin that doesn't support >=7 yet... |
Thanks. |
Hi all, anyone could compile properly with API 26? |
Hi all, just in case anyone is interested... in reality nothing is needed to be compliant with the Google Play store except adding to config.xml the two lines
No change in the rest and no need to update Cordova or Android platform.. So, as per today I confirm all works good with Android 8.0 and iOS 11.4 (iPhone 5S) with |
Note... only target version is needed, don't put also min-version at 26 or you will be able to serve only few devices... |
Hi,
I've just read this:
https://android-developers.googleblog.com/2017/12/improving-app-security-and-performance.html
In which it's mentioned that by August we will need to use a target SDK of 26
Currently it's 25...
Would that affect the functionality of this plugin being linked to a specific cordova and android version?
The text was updated successfully, but these errors were encountered: