-
Notifications
You must be signed in to change notification settings - Fork 917
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
Cordova 9 support #2581
Comments
I see. I will test it later. Thank you for letting me know. |
Please try to install from the multiple_maps branch. |
I had to downgrade to [email protected] to make it work on my system. Support for cordova 9 is pretty important. |
As I said above, please use the multiple_maps branch. |
I can confirm, the multiple_maps branch works. I'm actually not using the multiple_branchs branch, I use the published npm package. Is there a plan to make a npm release of this branch? Are there breaking changes? |
Ah sorry, too early: ios install still failes because of |
Please report with error log, not just |
Sorry, I don't have the call stack right now, but I can point you to the error. On line 3, 5 and 6 there are usages of the removed |
Give instructions on how to install the plugin on Cordova 9.0 and Android 8.0
|
Hello, |
PR is merged to the multiple_maps branch. |
What's with line 3? In the PR there is still the requireCordovaModule() present! Shouldn't this be replaced, too? |
No. |
IMHO @mariusbolik is right, that it should be replaced. but @wf9a5m75 is right too and the last PR solved the issue. Thank you. Since working with git references is slowing the restore process quite a bit: is there a plan to make a npm release too? |
Hey i have the same problem with ionic cordova plugin add cordova-plugin-geolocation |
@mariusbolik on line 3 the command @daninietoandres just use the master-branch of the |
On attempting to use the multiple_maps branch, i get the same result
im running cordova 9.0.0, ionic 4.12.0 & npm 6.9.0 |
@mariusbolik I think this is unrelated to this issue / plugin. Try uninstalling |
@mariusbolik I actually just came across this issue when upgrading |
@mariusbolik Please try the @adamduren's advice. If it does not work, please share your project files on Github repository. I will check it. |
@adamduren WOW! Thank you. This really was the solution! Everything works fine now. |
Update on the issue, simply downgrading to Cordova 8.1.0 worked for me as well, I believe downgrading angular would also do the trick |
@thineikhaing we're still waiting on @wf9a5m75 to release a new version I believe |
'cordova-plugin-googlemaps': CordovaError: Using "requireCordovaModule" to load non-cordova module'cordova-plugin-googlemaps': CordovaError: Using "requireCordovaModule" to load non-cordova module same here |
Use the multiple_maps branch |
@wf9a5m75 when you create new releases are they from |
I solved it by changing the version of Cordova. npm uninstall -g cordova Of causality will they have any improvement for cordova 9? |
Muchas gracias @luisgagocasas Su respuesta soluciono mi problema |
cordova-plugin-googlemaps v2.6.0 has released. |
Sorry but I have the problem with the 2.6.2 |
trying with 'npm i' is worked for me. |
I am getting this issue for Installing "cordova-plugin-enable-multidex" for android |
I am also currently experiencing the following issue on IOS when using Cordova 9, when trying to build my app:
My Ionic Info:
My plugins:
However, when I downgrade to Cordova 8.1.2 the application crashes and I get an error alert on the iPhone (on the device, not in Xcode) when launching the app: The Google Maps SDK for iOS crashes, when CFBundleExecutable contains Latin Characters Check CordovaGoogleMaps.m for more information. After entering the maps page once, the whole app crashes and is rendered unusable. I am using the multiple_maps branch like suggested here and would really appreciate any insights regarding this issue. If needed, I can create a new one. I have also followed the guide on https://ionic.zendesk.com/hc/en-us/articles/360009643533-iOS-Builds-fail-to-install-cordova-plugin-googlemaps-, which does not change anything in my case. |
First of all, please read this thread, and please please please post formatted text.
Then did you check
Please search this warning message before posting. And I think your cocoapod is old or too much latest. |
I am really sorry that I hadn't followed the guidelines and appreciate that you managed to take a look at it anyways!
I will check it out as soon as I can and get back to you!
My CocoaPods version:
|
For crying out loud, please find the past log. |
I'm submitting a ...
OS: (check one with "x")
The comming cordova 9 version will drop the support for
requireCordovaModule
calls in the hooks.Example error message with the latest nightlies from cordova:
The text was updated successfully, but these errors were encountered: