Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Cordova 9 support #133

Closed
wants to merge 0 commits into from
Closed

Cordova 9 support #133

wants to merge 0 commits into from

Conversation

pcsantana
Copy link
Contributor

What

Cordova 9 support! Fixes #132

Verification Steps

  1. npm install cordova@latest
  2. ionic cordova platform rm android
  3. Add this plugin (from forked repo to test)
  4. ionic cordova platform add android@latest

The build succeed!

Checklist:

  • Code has been tested locally by PR requester
  • Changes have been successfully verified by another team member

Progress

  • Finished task

Additional Notes

PS.: Sorry for the mess up the commits history. I commited the solution before I fetch the upstream repo, and I merged it. So it came with older commits. But actually, only the 783c65a commit fixes the issue.

@danielpassos
Copy link
Collaborator

Hey @pcsantana Can you rebase it to fix the commits problem?

You can run it on your master branch (assuming your git is origin and aerogear is upstream)
git rebase upstream/master

@pcsantana
Copy link
Contributor Author

I tried as suggested, but the commits history got worse. So, I reset it and committed again. Sorry for that.
I closed this in favor to a new pull request #134

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

Successfully merging this pull request may close these issues.

update to the latest version of Cordova
2 participants