ratson / cordova-plugin-admob-free

New development has been moved to "admob-plus-cordova", https://github.com/admob-plus/admob-plus/tree/master/packages/cordova
https://github.com/admob-plus/admob-plus
MIT License
495 stars 211 forks source link

Updating admob-sdk dependency to v0.14.0. #156

Open roshkesso opened 6 years ago

roshkesso commented 6 years ago

Hi @ratson,

I am using your plugin for both my iOS and Android app. (Thank you for your great work here). However I am having an issue with the iOS version only. I am just getting a blank white view. Debugging in xcode I am seeing the callback fire: (void)adView:(GADBannerView )view didFailToReceiveAdWithError:(GADRequestError )error with the error: Request Error: No ad to show.

I notice some other people are also seeing a similar issue here: https://forum.ionicframework.com/t/working-app-with-admob-free-plugin/110879/30

I am not 100% sure what the issue is but I am getting the warning that my AdMob SDK is out of date. I have contacted the SDK team and I am still waiting to hear from them. I have also tested my AdMob Unit Id with their Sample Banner app on my iOS device and it is working fine.

I noticed you updated the cordova-admob-sdk iOS SDK to v7.27.0 in v0.14.0 here: https://github.com/rehy/cordova-admob-sdk

I was wondering if there are plans to update this plugin to use v0.14.0? This will require the plugin config to also provide the Admob App Id. But I am thinking it might be required to make things work.....

If I hear back from the SDK team I will post any resolutions here.

Cheers, Rosh

PS. Sorry for the long winded post.

roshkesso commented 6 years ago

Just a quick update from a response from the Admob SDK team that the older SDK are still working fine so the issue wasn't with 7.26.

It would be great to get the latest SDK and also provide in the config a param for the App Id so you can initialize the SDK properly.

Cheers, Rosh