Skip to content

feat: rewarded ad sample app - #1897

Merged
peterporfy merged 10 commits into
mainfrom
ads-296-sample
Aug 20, 2026
Merged

feat: rewarded ad sample app#1897
peterporfy merged 10 commits into
mainfrom
ads-296-sample

Conversation

@peterporfy

@peterporfy peterporfy commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Adds a new adsTester sample app, which will potentially gain more ad related examples. First, I added it to the existing tester app, but it would have required multiple changes and extra dependencies which are probably not that relevant for most of the users, so I thought it is cleaner to have it separately for now.


Note

Low Risk
Changes are confined to a new example app and CI wiring; no production SDK or library code paths are modified.

Overview
Adds a new examples/adsTester React Native app that demonstrates RevenueCat’s rewarded-ad reward verification flow end to end with AdMob (react-native-google-mobile-ads), separate from the main purchase tester examples.

The sample wires generateRewardVerificationToken → AdMob serverSideVerificationOptionspollRewardVerification on EARNED_REWARD, with test AdMob IDs and a placeholder API key. react-native-purchases is resolved from repo src via Babel/Metro/react-native.config.js (same pattern as purchaseTesterTypescript), not as an npm dependency.

CircleCI gains adstester_ios and adstester_android jobs (root + example yarn install, Metro bundle, typecheck/lint on Android, native debug builds) and both are registered on the main build-test-hold-deploy workflow.

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

Copy link
Copy Markdown
Contributor Author

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

@RevenueCat-Danger-Bot

RevenueCat-Danger-Bot commented Aug 11, 2026

Copy link
Copy Markdown
2 Warnings
⚠️ Size check is being bypassed due to the presence of the label "danger-bypass-size-limit"
⚠️ Size increase: 455.69 KB

Generated by 🚫 Danger

@peterporfy peterporfy added the pr:other A code change that improves performance label Aug 11, 2026 — with Graphite App
@peterporfy
peterporfy marked this pull request as ready for review August 14, 2026 08:02
@peterporfy
peterporfy requested a review from a team as a code owner August 14, 2026 08:02
Comment thread examples/adsTester/android/app/build.gradle
Comment thread examples/adsTester/App.tsx

@AlvaroBrey AlvaroBrey 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.

Only a light review for this one since it's huge 🙃

Beside the below comments, please add CI for this. Ideally build both platforms on separate jobs, plus yarn build/lint

Comment thread examples/purchaseTesterTypescript/ios/Podfile.lock Outdated
Comment thread examples/adsTester/App.tsx
Comment thread examples/adsTester/App.tsx Outdated
Comment thread examples/adsTester/App.tsx Outdated
Comment thread examples/adsTester/App.tsx Outdated
@peterporfy
peterporfy force-pushed the ads-296-sample branch 2 times, most recently from d40f096 to ca58631 Compare August 20, 2026 08:42
Comment thread examples/adsTester/App.tsx

@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 c966889. Configure here.

Comment thread .circleci/config.yml Outdated
@peterporfy

Copy link
Copy Markdown
Contributor Author

@AlvaroBrey I've added a circleci config based on the expo example app

@AlvaroBrey AlvaroBrey 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.

Danger is failing because of PR size, you can just add the label suggested in the message in this case. You may need to rebase or push something so that Danger picks it up

Thanks for addressing the feedback! 🚢

@peterporfy peterporfy added the danger-bypass-size-limit Apply this label to bypass Dangerbot's size limit. label Aug 20, 2026
@peterporfy
peterporfy merged commit f4e235a into main Aug 20, 2026
13 checks passed
@peterporfy
peterporfy deleted the ads-296-sample branch August 20, 2026 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

danger-bypass-size-limit Apply this label to bypass Dangerbot's size limit. pr:other A code change that improves performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants