Skip to content

feat(ads): use the AdMob adapter in adsTester - #2033

Merged
peterporfy merged 2 commits into
mainfrom
porfy/react-native-admob-example
Oct 16, 2026
Merged

peterporfy merged 2 commits into
mainfrom
porfy/react-native-admob-example

Conversation

@peterporfy

@peterporfy peterporfy commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Description

Rewrites the adsTester example to use the react-native-purchases-admob adapter from #2023, split out so the adapter PR stays reviewable on its own. Stacked on #2023.

Changes

Examples

  • adsTester rewritten to use the adapter

Tests

  • full e2e simulator test with all ad formats, against production system, both with android and ios builds

Note

Low Risk
Changes are limited to the example app and its bundler/tsconfig wiring; no production SDK behavior is modified in this diff.

Overview
Rewrites the adsTester sample so it exercises react-native-purchases-admob instead of hand-rolling AdMob SSV tokens, polling, and a single rewarded-interstitial flow.

The UI becomes a scrollable dashboard: reusable FormatCard load/show cards for interstitial, rewarded, rewarded interstitial, and app open (plain showAndTrack vs showWithRewardVerification on rewarded formats), plus a bottom banner wired through useTrackingBannerAdProps. Manual uuid / generateRewardVerificationToken / pollRewardVerification code is removed in favor of adapter factories and show helpers.

Monorepo wiring mirrors react-native-purchases: Babel alias, TypeScript paths, and Metro watchFolders / extraNodeModules resolve the adapter from source and dedupe react-native-google-mobile-ads (including blocking the adapter workspace’s node_modules). Dependencies drop uuid and react-native-get-random-values and bump react-native-google-mobile-ads to ^17.

Reviewed by Cursor Bugbot for commit f3a852d. Bugbot is set up for automated code reviews on this repo. Configure here.

peterporfy commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@peterporfy
peterporfy force-pushed the porfy/react-native-admob-example branch 4 times, most recently from d0b681e to 089cb87 Compare October 15, 2026 13:17
@peterporfy
peterporfy marked this pull request as ready for review October 15, 2026 13:18
@peterporfy
peterporfy requested a review from a team as a code owner October 15, 2026 13:18
@peterporfy
peterporfy force-pushed the porfy/react-native-admob-example branch from 089cb87 to fcf7fab Compare October 15, 2026 13:24

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread examples/adsTester/App.tsx
@peterporfy
peterporfy force-pushed the porfy/react-native-admob-example branch 3 times, most recently from 1282d10 to 6e844ae Compare October 15, 2026 14:34
@peterporfy
peterporfy force-pushed the porfy/react-native-admob-adapter branch from ff5ef82 to ed74ab0 Compare October 15, 2026 14:34

@tonidero tonidero left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looking great!

@peterporfy
peterporfy force-pushed the porfy/react-native-admob-example branch from 6e844ae to 10a1ea9 Compare October 15, 2026 15:53

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread examples/adsTester/App.tsx
@peterporfy
peterporfy force-pushed the porfy/react-native-admob-example branch 3 times, most recently from 3485fb5 to 6e765e2 Compare October 16, 2026 07:36
@peterporfy peterporfy added the pr:other A code change that improves performance label Oct 16, 2026
@peterporfy
peterporfy force-pushed the porfy/react-native-admob-example branch from 5eeae86 to bddb5e5 Compare October 16, 2026 08:44

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ 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 bddb5e5. Configure here.

Comment thread examples/adsTester/App.tsx
Base automatically changed from porfy/react-native-admob-adapter to main October 16, 2026 08:58
@peterporfy
peterporfy force-pushed the porfy/react-native-admob-example branch from bddb5e5 to f3a852d Compare October 16, 2026 09:00
@peterporfy
peterporfy merged commit 2591315 into main Oct 16, 2026
17 checks passed
@peterporfy
peterporfy deleted the porfy/react-native-admob-example branch October 16, 2026 09:29
RCGitBot added a commit that referenced this pull request Oct 19, 2026
**This is an automatic release.**

## RevenueCat SDK
### ✨ New Features
* feat(ads): admob adapter (#2023) via Peter Porfy (@peterporfy)

### 🔄 Other Changes
* feat(ads): use the AdMob adapter in adsTester (#2033) via Peter Porfy
(@peterporfy)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Mechanical version and changelog updates only; functional AdMob work
is assumed to have landed in earlier PRs referenced by the changelog.
> 
> **Overview**
> **Automatic release** that publishes **10.15.0** across the monorepo
(`react-native-purchases`, `react-native-purchases-admob`, UI, Store
Galaxy) and aligns reported plugin versions on iOS, Android, and the
browser shim.
> 
> The diff is almost entirely mechanical: `.version` / `package.json` /
Gradle `versionName`, `PLUGIN_VERSION` and `platformFlavorVersion`,
`yarn.lock` peer pins, `CHANGELOG.md` / `CHANGELOG.latest.md`, a new
**10.15.0** row in `VERSIONS.md` (same hybrid-common and native SDK pins
as 10.14.0), and the docs redirect to **10.15.0**. No AdMob adapter
implementation appears here; the release notes attribute user-facing
changes to the **AdMob adapter** (#2023) and wiring it in **adsTester**
(#2033).
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
a980621. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

---------

Co-authored-by: RevenueCat CI <dev+ci@revenuecat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:other A code change that improves performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants