feat(admob): demonstrate banner ads - #4095
Merged
Merged
Conversation
This was referenced Aug 26, 2026
polmiro
force-pushed
the
next-gen-admob-sample-banner
branch
from
August 27, 2026 08:53
687e997 to
28581e9
Compare
polmiro
marked this pull request as ready for review
August 27, 2026 08:59
polmiro
force-pushed
the
next-gen-admob-sample-banner
branch
from
August 28, 2026 07:59
28581e9 to
cb23ea3
Compare
polmiro
force-pushed
the
next-gen-admob-sample-banner
branch
from
August 28, 2026 12:20
cb23ea3 to
4c5a2c8
Compare
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 4c5a2c8. Configure here.
AlvaroBrey
reviewed
Aug 31, 2026
polmiro
force-pushed
the
next-gen-admob-sample-banner
branch
from
September 1, 2026 08:12
4c5a2c8 to
09fc0d5
Compare
📸 Snapshot Test1 modified, 620 unchanged
🛸 Powered by Emerge Tools |
Base automatically changed from
next-gen-admob-sample-preloader-ui
to
next-gen-admob-sdk
September 1, 2026 08:46
polmiro
force-pushed
the
next-gen-admob-sample-banner
branch
from
September 1, 2026 08:46
09fc0d5 to
1035d0b
Compare
AlvaroBrey
approved these changes
Sep 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Expands the Google Mobile Ads Next-Gen example app with banner workflows:
Screenshots
Note
Low Risk
Sample-app and documentation changes only; no production SDK or auth/data-path changes.
Overview
Replaces the Banner placeholder in the AdMob Next-Gen sample with a working Banner screen that shows RevenueCat-tracked banner flows end to end.
Direct mode loads into an
AdViewvialoadAndTrackAd, with status for load success/failure plus impression, click, and refresh callbacks. Preloaded mode uses the shared preloader UI to start/stop buffering, then poll + register a buffered ad on the same view with distinct placements (banner_direct,banner_preload,banner_poll). Switching load modes recreates theAdView(key(mode)) and resets status so banner state does not leak across modes.The sample README Coverage line for banners is updated to describe lifecycle/refresh callbacks instead of callback-replacement wording.
Reviewed by Cursor Bugbot for commit 1035d0b. Bugbot is set up for automated code reviews on this repo. Configure here.