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

fix: interstitials and rewarded ads #18

Merged

Conversation

dylancom
Copy link
Collaborator

@dylancom dylancom commented Dec 6, 2021

Description

Fixes #6 & #16

  • Make it work on iOS
  • Make it work on Android

@docs-page
Copy link

docs-page bot commented Dec 6, 2021

To view this pull requests documentation preview, visit the following URL:

docs.page/invertase/react-native-google-ads~18

Documentation is deployed and generated using docs.page.

@codecov
Copy link

codecov bot commented Dec 6, 2021

Codecov Report

Merging #18 (6d42eb9) into main (7c7c8b5) will increase coverage by 0.26%.
The diff coverage is 31.25%.

❗ Current head 6d42eb9 differs from pull request most recent head da5eeb0. Consider uploading reports for the commit da5eeb0 to get more accurate results

@@            Coverage Diff             @@
##             main      #18      +/-   ##
==========================================
+ Coverage   18.95%   19.21%   +0.26%     
==========================================
  Files          21       21              
  Lines         644      651       +7     
  Branches      156      158       +2     
==========================================
+ Hits          122      125       +3     
- Misses        405      407       +2     
- Partials      117      119       +2     

Copy link
Collaborator

@mikehardy mikehardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great - will merge pending those comment lines
I haven't tested but I know this is an improvement and I know you are all over this, so just going to merge as soon as the annotations for inclusive language are nailed

Thanks!

Copy link
Collaborator

@mikehardy mikehardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

boom - let's go :-)

@mikehardy mikehardy merged commit 9569b98 into invertase:main Dec 7, 2021
github-actions bot pushed a commit that referenced this pull request Dec 8, 2021
# 1.0.0 (2021-12-08)

### Bug Fixes

* android app id from app.json works, example has detox integrated and working in android ([24f83b2](24f83b2))
* **android:** use correct module name for banner ads ([#17](#17)) ([7c7c8b5](7c7c8b5))
* displaying banner ads, config in docs ([#14](#14)) ([21e4f42](21e4f42))
* incorrect module naming ([#10](#10)) ([79f86b9](79f86b9))
* interstitials and rewarded ads ([#18](#18)) ([9569b98](9569b98))
* **ios:** add ios config scripts missed in previous commit ([227ca7c](227ca7c))
* **ios:** ios app.json --> Info.plist auto-config works now ([6c1f4d4](6c1f4d4))
* **ios:** missing dollar signs in podspec version override ([#8](#8)) ([5984288](5984288))
* **release:** remove npm config and git config as well ([8139e88](8139e88))
* **release:** semantic release does not need the ssh key agent config ([930bc03](930bc03))
* remove incorrect git branch ref in publish workflow, clean changelog ([aec5b84](aec5b84))
* **test:** link to jet package via named branch vs commit hash ([#7](#7)) ([151f6aa](151f6aa))

### Features

* example test in new test harness works ([2041fa2](2041fa2))
* **native:** native code extraction is complete and passes static type check ([02c2af6](02c2af6))
Star-dev325 added a commit to Star-dev325/react-native-google-mobile-ads that referenced this pull request Jun 7, 2024
# 1.0.0 (2021-12-08)

### Bug Fixes

* android app id from app.json works, example has detox integrated and working in android ([24f83b2](invertase/react-native-google-mobile-ads@24f83b2))
* **android:** use correct module name for banner ads ([#17](invertase/react-native-google-mobile-ads#17)) ([7c7c8b5](invertase/react-native-google-mobile-ads@7c7c8b5))
* displaying banner ads, config in docs ([#14](invertase/react-native-google-mobile-ads#14)) ([21e4f42](invertase/react-native-google-mobile-ads@21e4f42))
* incorrect module naming ([#10](invertase/react-native-google-mobile-ads#10)) ([79f86b9](invertase/react-native-google-mobile-ads@79f86b9))
* interstitials and rewarded ads ([#18](invertase/react-native-google-mobile-ads#18)) ([9569b98](invertase/react-native-google-mobile-ads@9569b98))
* **ios:** add ios config scripts missed in previous commit ([227ca7c](invertase/react-native-google-mobile-ads@227ca7c))
* **ios:** ios app.json --> Info.plist auto-config works now ([6c1f4d4](invertase/react-native-google-mobile-ads@6c1f4d4))
* **ios:** missing dollar signs in podspec version override ([#8](invertase/react-native-google-mobile-ads#8)) ([5984288](invertase/react-native-google-mobile-ads@5984288))
* **release:** remove npm config and git config as well ([8139e88](invertase/react-native-google-mobile-ads@8139e88))
* **release:** semantic release does not need the ssh key agent config ([930bc03](invertase/react-native-google-mobile-ads@930bc03))
* remove incorrect git branch ref in publish workflow, clean changelog ([aec5b84](invertase/react-native-google-mobile-ads@aec5b84))
* **test:** link to jet package via named branch vs commit hash ([#7](invertase/react-native-google-mobile-ads#7)) ([151f6aa](invertase/react-native-google-mobile-ads@151f6aa))

### Features

* example test in new test harness works ([2041fa2](invertase/react-native-google-mobile-ads@2041fa2))
* **native:** native code extraction is complete and passes static type check ([02c2af6](invertase/react-native-google-mobile-ads@02c2af6))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[🐛] 🔥 Invariant Violation: new NativeEventEmitter() requires a non-null argument.
2 participants