Skip to content
This repository has been archived by the owner on Jan 14, 2022. It is now read-only.

Fix for newer Android version #112

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

pcotte
Copy link

@pcotte pcotte commented Jun 7, 2019

Modified to support new version of Cordova Android-Platform

@msftclas
Copy link

msftclas commented Jun 7, 2019

CLA assistant check
All CLA requirements met.

@SamLoy
Copy link

SamLoy commented Jul 17, 2019

What needs to happen to get this PR merged in?

This plugin currently doesn't work with cordova@9.0.0 since they removed "requireCordovaModule" apache/cordova#85

Using this PR worked for me against cordova@9.0.0

@santiq
Copy link

santiq commented Aug 7, 2020

This would solve the problem:

Updating Cordova configuration from W3C manifest...
Using "requireCordovaModule" to load non-cordova module "q" is not supported. Instead, add this module to your dependencies and use regular "require" to load it.

@f2bo can you take a look at this PR?

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.

4 participants