-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Clarification on PhoneGap Build Service #961
Comments
up! |
1 similar comment
up! |
Seriously. The docs are way too vague here. |
Anyone figured out how to use this plugin in Phonegap ? |
@mobileDev2020 try using my fork : https://github.com/freefiona85/cordova-plugin-firebase - it works on current Firebase version and Phonegap Build. You can see quite a lot of changes and debugs there. some function was disabled temporarily, but many of the functions are still usable. |
@freefiona85 , thanks for this update. I tried installing this plugin in my project, along with the google-services plugin as required by the firebase. I get a build error saying both these plugins are trying to install the same file google-service-Info.plist . I commented out the google-services plugin , but the build complains the googleservice-info is already present at the destination. If I understand the working of firebase, I am required to manually drop by project services file in the code folder. Please correct me if I am missing something. |
@mobileDev2020 we don't need to install google-services plugin anymore now as far as i tried. Also, try removing from your config.xml. it should work because it was copied via plugin (line 87, plugin.xml). I think it's more versatile if i remove that line from the plugin later, though. |
@freefiona85 , thanks for the prompt response. I commented out services plugin from config.xml already. I have the googleservice-info file in my project that ties my project to analytics picked up by the firebase plugin. I dont understand why phonegap build says - Failed to install 'cordova-plugin-firebase': CordovaError: Uh oh! |
oh! i see the comment section deleted what i should write! I just push a new commit on my fork, removing that line on plugin.xml. Try rebuilding with the latest commit, it should remove the problem. |
That worked. |
That was weird. Any logs? can you debug it? |
@freefiona85 thanks for helping out. I tried using your plugin, without any other plugin to make sure there's no incompatibilities. It compiles on iOS (I haven't checked if it works), but it does not compile on Android. Here the config.xml:
Here's the error:
Are you using a different cli or config file? You also said that some function was disabled temporarily. Which function(s) is that? |
@Heshyo Oooh, there's an update to build gradle dependency few days ago. I updated the plugin so i froze the dependency to 3.5.1. Just tested myself with a clean plugin, it works with PGB. Also, some feature that's disabled are :
|
@freefiona85 Thanks a lot for the fast update! It now compiles on both iOS and Android, with all my necessary plugins. I haven't tried running it yet though, so I can't confirm everything is working. Maybe you could update the readme to show it works on Phonegap Build straight, and the currently disabled features (which I don't use yet). In case anyone uses cordova-plugin-x-socialsharing, I had the following issue when adding this plugin:
but there's already a pull request to fix this. So just using that other repo fixes the build. |
Readme and Travis CI updated, along with some edits. Thank you @Heshyo ! Also, yeah, x-socialsharing has a lot of problem with AndroidX build method too. Glad you found out how to fix it! |
I am able to build successfully on iOS but app crashes , doesn't open. Any tips on how to debug ? Anyone else seeing this ? Thanks! |
@mobileDev2020 Have you debugged it using Browser web inspector (Safari one)? To make sure what causes it to crash on the phone. |
@freefiona85, I have tried safari , but it doesnt go that far, I am guessing error is before app loads. |
@mobileDev2020 Have you tried xcode device logs? Or Console.app ? http://osxdaily.com/2018/07/12/how-view-ios-logs-on-mac/ |
@freefiona85, I will try the above approach. But these logs are not easy to read through, is there anything in particular I can look for, to understand the root cause of crash. |
@freefiona85, this console.app is a great option, thank you for that! I see that this issue was fixed long back in Wondering why it is showing up again. Anyone else seeing this ? |
@freefona85, I got the above error when I added I get this error at startup and app crashes |
@hesyo, Can you confirm your app works with the freefiona85's fork ? |
@freefiona85, But my app has links to open up outside web pages, say , google.com, also hrefs with "tel" . |
Any log error on clicking the outside link, @Heshyo ? As far as i know, there are nothing that should prevent those. Did you test it on real device or simulator? Also https://stackoverflow.com/questions/33339419/phone-links-are-not-working-inside-iframe-though-it-does-work-in-div-in-ios9-web |
Also it's for @mobileDev2020 , sorry. |
@freefion85 , I am using a device to test. These links and call to tel have been working with the google analytics plugin. The only change I am making is to use the firebase plugin to support analytics due to the Oct 31st deadline for Google Analytics. |
@freefiona85 , I tried with a lower of cli too. The links are definitely getting blocked. This plugin and the whitelist plugin are not working together inspite of tags like
Is anyone else seeing these issues? |
@freefiona85, I was able to get past these errors, html issues turns out. TIA for all your time! |
@mobileDev2020 yes, you can enable debugview mode on your device. https://firebase.google.com/docs/analytics/debugview this should help. Also, if you use logEvent, it should be able to be viewed on debugMode (along with all automatic logs like install, uninstall, open first time, and screens, for example). |
@freefiona85, thank you ! |
Anyone got ITMS 90078 warning when you pushed your app to the AppStore ? As I said before, this plugin is the only change in my new version and we got this warning . It doesn't stop us from publishing, but since my app does not use Push Notifications, I am not clear why it asks me to. |
This is not a bug on the latest version, but the documentation could use a bit more clarification on the phonegap build part. How should I declare my app ID and API Key in the xml? What tag should I use? I can't find any leads from the source code.
The text was updated successfully, but these errors were encountered: