Skip to content
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

Using with cordova android version 8.x.x on API SDK 28 #371

Closed
itshazlan opened this issue Sep 19, 2019 · 7 comments
Closed

Using with cordova android version 8.x.x on API SDK 28 #371

itshazlan opened this issue Sep 19, 2019 · 7 comments

Comments

@itshazlan
Copy link

itshazlan commented Sep 19, 2019

I want to use NFC plugin for my Ionic 3 project using android version 8.1.0 on API SDK 28.

Because when uploading app into the Google Play Store require the latest API SDK 28, and the NFC plugin gives me the error when installing on cordova android version 8.X.X.

It says Error: Unable to graft xml at selector "/manifest/uses-sdk" from "platforms\android\app\src\main\AndroidManifest.xml" during config install at ConfigFile_graft_child [as graft_child]

Please, someone, help me how to troubleshoot the plugin compatibility. Thank you.

@itshazlan itshazlan changed the title Using with platform android version 8.x.x on API SDK 28 Using with cordova android version 8.x.x on API SDK 28 Sep 19, 2019
@flossy
Copy link

flossy commented Sep 19, 2019

+1

at projectRoot/plugins/phonegap-nfc/plugin.xml line: 39 comment out

<!--edit-config file="AndroidManifest.xml" target="/manifest/uses-sdk" mode="merge">
  <uses-sdk android:minSdkVersion="19" />
</edit-config-->

passed $ cordova platform add android

Maybe the minVersion added in config.xml is enough.
For your information...

@sc0ttdav3y
Copy link

+1

I moved <uses-sdk android:minSdkVersion="19" /> into the <config-file target="AndroidManifest.xml" parent="/manifest"> block above and it works again.

@khayargoli
Copy link

khayargoli commented Sep 21, 2019

I have the same problem, I am using phonegap build. Please have a fix for this on the plugin itself
:)

@mobidev111
Copy link

Could anyone of you open a pull request with this fix? then we can ping the maintainer to release a new version.

@cesardmoro
Copy link

any fix on this ? i have to edit plugin.xml file but this will be broken on CI

@LuisEGR
Copy link
Contributor

LuisEGR commented Sep 24, 2019

Fixed here: #372

If you need to use this fix, I uploaded it to NPM so you can use it until the issue gets merged:

npm i phonegap-nfc-fxmanifest

or

ionic cordova plugin add phonegap-nfc-fxmanifest

@don
Copy link
Member

don commented Oct 18, 2019

#372 should fix this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants