-
-
Notifications
You must be signed in to change notification settings - Fork 154
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
feat(android): potentially improves mediation performance for banners #527
Conversation
To view this pull requests documentation preview, visit the following URL: docs.page/invertase/react-native-google-mobile-ads~527 Documentation is deployed and generated using docs.page. |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #527 +/- ##
=======================================
Coverage 43.66% 43.66%
=======================================
Files 29 29
Lines 536 536
Branches 147 147
=======================================
Hits 234 234
Misses 302 302 |
@birdofpreyru ! Sorry for the delay on this and anything else you've posted, haven't had any time lately. Will spin through them shortly and collaborate + work to get these merged as promised. Thanks for your patience |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry this took so long to review, sincere apologies
Code looks fine
## [12.11.0](v12.10.0...v12.11.0) (2024-02-19) ### Features * **android:** potentially improves mediation performance for banners ([#527](#527)) ([32f550a](32f550a)) ### Bug Fixes * **consent:** the package returns the wrong value for consent for 'Create a personalized content profile' purpose ([7aef863](7aef863))
🎉 This PR is included in version 12.11.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
## [12.11.0](invertase/react-native-google-mobile-ads@v12.10.0...v12.11.0) (2024-02-19) ### Features * **android:** potentially improves mediation performance for banners ([#527](invertase/react-native-google-mobile-ads#527)) ([32f550a](invertase/react-native-google-mobile-ads@32f550a)) ### Bug Fixes * **consent:** the package returns the wrong value for consent for 'Create a personalized content profile' purpose ([7aef863](invertase/react-native-google-mobile-ads@7aef863))
Description
A little refactoring of the banner initialization code for Android, motivated by AdMob docs for mediation.
Related issues
Fixes #506
Release Summary
Potentially improves mediation performance for banners on Android.
Checklist
and followed the process outlined there for submitting PRs.
Android
iOS
e2e
tests added or updated in__tests__e2e__
jest
tests added or updated in__tests__
Test Plan
I ran the example app after update, it works fine. The change is also quite trivial.
Think
react-native-google-mobile-ads
is great? Please consider supporting the project with any of the below:Invertase
on Twitter