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: incorrect module naming #10

Merged

Conversation

dylancom
Copy link
Collaborator

@dylancom dylancom commented Dec 3, 2021

Description

This fixes #9 and makes the package usable again. With this changes I was able to render a banner ad again.

@docs-page
Copy link

docs-page bot commented Dec 3, 2021

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

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

Documentation is deployed and generated using docs.page.

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.

Nice!

@mikehardy
Copy link
Collaborator

The CI will likely fail (I'm not sure what state I left it in...) and I will likely merge this anyway - running them more out of curiosity then a gating need for green just right now

@mikehardy mikehardy added the workflow: pending merge Waiting on CI or question responses to merge, but otherwise ready label Dec 3, 2021
@mikehardy mikehardy merged commit 79f86b9 into invertase:main Dec 3, 2021
@mikehardy mikehardy removed the workflow: pending merge Waiting on CI or question responses to merge, but otherwise ready label Dec 3, 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: Module AppRegistry is not a registered callable module (calling runApplication)
2 participants