Skip to content
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

Content Rating API #328

Open
wants to merge 101 commits into
base: 1.x
Choose a base branch
from
Open

Content Rating API #328

wants to merge 101 commits into from

Conversation

enactium
Copy link

Hi,
I have an application targeted for G [3+] populations (cf. https://support.google.com/admob/answer/7562142?hl=en&ref_topic=7384665) that use admob SDK for providing some ads (Banner and Interstitial).

After submitting the app to google play, it was rejected because it should include instructions like this :
AdRequest request = new AdRequest.Builder().tagForChildDirectedTreatment(true).build();
adView.loadAd(request);

But, this SDK don’t provide any way (as specifying the emulator devices) to set such configurations,

Could you please add this new requirements to your SDK?

Regards.

please note, these changes are not backwards compatible.

The exported view properties now match the properties of the DFPBannerView.

I've now implemented the `validAdSizes` property, and replaced the `testDeviceID` property with `testDevices`. Also the simulator device id can be specified using `PublisherBanner.simulatorId` instead of `"EMULATOR"`.

The conversion of json to GADAdSize is now implemented using a RCTConvert category.

And lastly, the `loadBanner` method is now exposed to react, to allow easy banner refresh, and removes the need of checking which properties are available before making a request to admob.
this omits the need of explicit conversion
also, there are no nativeOnly events so removed those
instead of the mixed names like RNAdMobManager and BannerView they're all named like the implemented class, e.g. RNGADBannerView and RNGADBannerViewManager.
replaced deprecated RCTEventDispatcher with RCTEventEmitter
replaced deprecated RCTEventDispatcher with RCTEventEmitter
this is more or less the bare minimum, we should probably add some rules along the way
almostintuitive and others added 12 commits September 16, 2019 00:39
…ded() is called before requestAd() (#241)

* Fix: make sure we only invoke requestPromise once

* Fix: check if mRewardedVideoAd is not null before calling .isLoaded()

* remove newline
when using React Native AdMob with cocoapods, you have to add `Google-Mobile-Ads-SDK` to your `Podfile` manually, because that framework contains a binary framework, which isn't supported when used with `use_frameworks!`.
this updates the example app to use cocoapods to integrate all libraries
update example app using latest "react-native init" (0.60.5)
The UITests have been updated to assert ads are rendered. Something similar has to be done for Android implementation.
add GitHub workflow that sets up a React Native project with the library
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet