feat: rewarded ad sample app - #1897
Conversation
Generated by 🚫 Danger |
716265c to
eddc627
Compare
eddc627 to
cf7ec8f
Compare
AlvaroBrey
left a comment
There was a problem hiding this comment.
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
cf7ec8f to
90f2ff9
Compare
d40f096 to
ca58631
Compare
ca58631 to
75b4792
Compare
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 c966889. Configure here.
c966889 to
35b3312
Compare
35b3312 to
ba3f223
Compare
|
@AlvaroBrey I've added a circleci config based on the expo example app |
AlvaroBrey
left a comment
There was a problem hiding this comment.
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! 🚢


Adds a new
adsTestersample 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/adsTesterReact 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→ AdMobserverSideVerificationOptions→pollRewardVerificationonEARNED_REWARD, with test AdMob IDs and a placeholder API key.react-native-purchasesis resolved from reposrcvia Babel/Metro/react-native.config.js(same pattern aspurchaseTesterTypescript), not as an npm dependency.CircleCI gains
adstester_iosandadstester_androidjobs (root + example yarn install, Metro bundle, typecheck/lint on Android, native debug builds) and both are registered on the mainbuild-test-hold-deployworkflow.Reviewed by Cursor Bugbot for commit ba3f223. Bugbot is set up for automated code reviews on this repo. Configure here.