-
Notifications
You must be signed in to change notification settings - Fork 3.4k
[in_app_purchase_android] GooglePlayPurchaseParam add possibility set selected offerToken #8452
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
[in_app_purchase_android] GooglePlayPurchaseParam add possibility set selected offerToken #8452
Conversation
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.
This completely fell off my review queue, I'm sorry for the delay here.
This mostly LGTM, but before we can land can you
- resolve the conflicts
- Fix the format failure (you can see this either from the logs of the failing
Linux repo_checks
check (link1 -> link2) or by following the formatting readme (basically justdart format
the dart code here). - Update the CHANGELOG to follow the style guide https://github.com/flutter/flutter/blob/master/docs/ecosystem/contributing/README.md#changelog-style (maybe something like "Adds support for setting the
offerToken
onGooglePlayPurchaseParam
.)
@rmasarovic Are you still working on this PR? I can take over making the changelog changes if not, let me know. |
@rmasarovic I've updated the PR according to my comments as it appeared you were no longer working on it, hopefully that is ok! Will get this reviewed by other team members, to avoid reviewing my own changes. |
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.
ok
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.
LGTM
}); | ||
|
||
/// The 'changeSubscriptionParam' containing information for upgrading or | ||
/// downgrading an existing subscription. | ||
final ChangeSubscriptionParam? changeSubscriptionParam; | ||
|
||
/// For One-time product, "offerToken" shouldn't be filled. |
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.
nit: Add a space after first sentence.
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.
added a line break
autosubmit label was removed for flutter/packages/8452, because This PR has not met approval requirements for merging. The PR author is not a member of flutter-hackers and needs 1 more review(s) in order to merge this PR.
|
…bility set selected offerToken (flutter/packages#8452)
flutter/packages@1765c95...2bac766 2025-06-03 [email protected] [in_app_purchase_android] GooglePlayPurchaseParam add possibility set selected offerToken (flutter/packages#8452) 2025-06-03 [email protected] [camera] 🐛 Fix toggles overflow in the camera example (flutter/packages#9274) 2025-06-03 [email protected] [shared_preferences] Remove duplicate integration test (flutter/packages#9368) 2025-06-03 [email protected] [pointer_interceptor_web] Fix unresponsive input above PointerInterceptor on Safari and Firefox. (flutter/packages#9362) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages-flutter-autoroll Please CC [email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
… selected offerToken (flutter#8452) Option to add purchase parameters in GooglePlayPurchaseParam to set selected offer token. In subscribtion with multiple offer in base plan, the user can choose which offer to want buy and then this selected offer token is sent to the billing flow flutter/flutter#150348
Option to add purchase parameters in GooglePlayPurchaseParam to set selected offer token. In subscribtion with multiple offer in base plan, the user can choose which offer to want buy and then this selected offer token is sent to the billing flow
flutter/flutter#150348
Pre-launch Checklist
dart format
.)[shared_preferences]
pubspec.yaml
with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.CHANGELOG.md
to add a description of the change, following repository CHANGELOG style, or this PR is exempt from CHANGELOG changes.///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.