Skip to content
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] Introduced new ReplacementMode for Android's billing client #6515

Merged
merged 9 commits into from
May 22, 2024

Conversation

vongrad
Copy link
Contributor

@vongrad vongrad commented Apr 14, 2024

Introduced new ReplacementMode for Android's billing client and deprecated ProrationMode.

This PR is a follow-up on #6403, where it was decided that we should not replace the ProrationMode with ReplacementMode, but instead only deprecate ProrationMode. The reason for a new PR is also that in_app_purchase_android version 0.3.3 changed internal platform communication to Pigeon, which meant I had to make major changes to my original PR.

List which issues are fixed by this PR. You must list at least one issue.
flutter/flutter#128957

Pre-launch Checklist

  • I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • I read and followed the [relevant style guides] and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/packages repo does use dart format.)
  • I signed the [CLA].
  • The title of the PR starts with the name of the package surrounded by square brackets, e.g. [shared_preferences]
  • I [linked to at least one issue that this PR fixes] in the description above.
  • I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy], or this PR is [exempt from version changes].
  • I updated CHANGELOG.md to add a description of the change, [following repository CHANGELOG style].
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is [test-exempt].
  • All existing and new tests are passing.

@vongrad
Copy link
Contributor Author

vongrad commented Apr 20, 2024

@bparrishMines I have made this PR in favor of #6403 (comment). Any chance it can be reviewed?

@yaakovschectman yaakovschectman changed the title Introduced new ReplacementMode for Android's billing client [in_app_purchase_android] Introduced new ReplacementMode for Android's billing client May 2, 2024
Copy link
Contributor

@bparrishMines bparrishMines left a comment

Choose a reason for hiding this comment

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

LGTM

@gmackall could you give this a secondary review

Copy link
Member

@gmackall gmackall left a comment

Choose a reason for hiding this comment

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

LGTM minus the nits @bparrishMines requested

@vongrad
Copy link
Contributor Author

vongrad commented May 10, 2024

Anything missing from my end?

@vongrad
Copy link
Contributor Author

vongrad commented May 13, 2024

@bparrishMines I have made the changes you requested, could you review it?

Copy link
Contributor

@bparrishMines bparrishMines left a comment

Choose a reason for hiding this comment

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

LGTM

@bparrishMines bparrishMines added the autosubmit Merge PR when tree becomes green via auto submit App label May 22, 2024
@auto-submit auto-submit bot merged commit 0bdda37 into flutter:main May 22, 2024
74 checks passed
TecHaxter pushed a commit to TecHaxter/flutter_packages that referenced this pull request May 22, 2024
…s billing client (flutter#6515)

Introduced new `ReplacementMode` for Android's billing client and deprecated `ProrationMode`.

This PR is a follow-up on [https:/flutter/packages/pull/6403](https:/flutter/packages/pull/6403), where it was decided that we should not replace the `ProrationMode` with `ReplacementMode`, but instead only deprecate `ProrationMode`. The reason for a new PR is also that `in_app_purchase_android` version `0.3.3` changed internal platform communication to Pigeon, which meant I had to make major changes to my original PR.

*List which issues are fixed by this PR. You must list at least one issue.*
flutter/flutter#128957
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 22, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request May 22, 2024
flutter/packages@ba19b24...6525441

2024-05-22 [email protected] [local_auth] Convert native unit tests to Swift (flutter/packages#6779)
2024-05-22 49699333+dependabot[bot]@users.noreply.github.com [interactive_media_ads]: Bump androidx.annotation:annotation from 1.5.0 to 1.8.0 in /packages/interactive_media_ads/android (flutter/packages#6771)
2024-05-22 [email protected] [in_app_purchase_android] Introduced new ReplacementMode for Android's billing client (flutter/packages#6515)
2024-05-21 [email protected] [go_router] New feature improve debug full path (flutter/packages#6714)
2024-05-21 [email protected] [interactive_media_ads] Add SPM support (flutter/packages#6756)
2024-05-21 [email protected] Roll Flutter from 02a6c91 to d02292d (22 revisions) (flutter/packages#6778)
2024-05-21 [email protected] [local_auth] Remove use of OCMock (flutter/packages#6757)
2024-05-21 49699333+dependabot[bot]@users.noreply.github.com Bump github/codeql-action from 3.25.3 to 3.25.6 (flutter/packages#6777)
2024-05-20 49699333+dependabot[bot]@users.noreply.github.com [file_selector]: Bump androidx.annotation:annotation from 1.7.1 to 1.8.0 in /packages/file_selector/file_selector_android/android (flutter/packages#6769)
2024-05-20 49699333+dependabot[bot]@users.noreply.github.com [in_app_pur]: Bump androidx.annotation:annotation from 1.7.1 to 1.8.0 in /packages/in_app_purchase/in_app_purchase_android/android (flutter/packages#6765)
2024-05-20 49699333+dependabot[bot]@users.noreply.github.com [url_launcher]: Bump androidx.annotation:annotation from 1.7.1 to 1.8.0 in /packages/url_launcher/url_launcher_android/android (flutter/packages#6762)
2024-05-20 [email protected] Roll Flutter from adf279f to 02a6c91 (8 revisions) (flutter/packages#6776)

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],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https:/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
victorsanni pushed a commit to victorsanni/flutter that referenced this pull request May 31, 2024
flutter/packages@ba19b24...6525441

2024-05-22 [email protected] [local_auth] Convert native unit tests to Swift (flutter/packages#6779)
2024-05-22 49699333+dependabot[bot]@users.noreply.github.com [interactive_media_ads]: Bump androidx.annotation:annotation from 1.5.0 to 1.8.0 in /packages/interactive_media_ads/android (flutter/packages#6771)
2024-05-22 [email protected] [in_app_purchase_android] Introduced new ReplacementMode for Android's billing client (flutter/packages#6515)
2024-05-21 [email protected] [go_router] New feature improve debug full path (flutter/packages#6714)
2024-05-21 [email protected] [interactive_media_ads] Add SPM support (flutter/packages#6756)
2024-05-21 [email protected] Roll Flutter from 02a6c91 to d02292d (22 revisions) (flutter/packages#6778)
2024-05-21 [email protected] [local_auth] Remove use of OCMock (flutter/packages#6757)
2024-05-21 49699333+dependabot[bot]@users.noreply.github.com Bump github/codeql-action from 3.25.3 to 3.25.6 (flutter/packages#6777)
2024-05-20 49699333+dependabot[bot]@users.noreply.github.com [file_selector]: Bump androidx.annotation:annotation from 1.7.1 to 1.8.0 in /packages/file_selector/file_selector_android/android (flutter/packages#6769)
2024-05-20 49699333+dependabot[bot]@users.noreply.github.com [in_app_pur]: Bump androidx.annotation:annotation from 1.7.1 to 1.8.0 in /packages/in_app_purchase/in_app_purchase_android/android (flutter/packages#6765)
2024-05-20 49699333+dependabot[bot]@users.noreply.github.com [url_launcher]: Bump androidx.annotation:annotation from 1.7.1 to 1.8.0 in /packages/url_launcher/url_launcher_android/android (flutter/packages#6762)
2024-05-20 [email protected] Roll Flutter from adf279f to 02a6c91 (8 revisions) (flutter/packages#6776)

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],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https:/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
arc-yong pushed a commit to Arctuition/packages-arc that referenced this pull request Jun 14, 2024
…s billing client (flutter#6515)

Introduced new `ReplacementMode` for Android's billing client and deprecated `ProrationMode`.

This PR is a follow-up on [https:/flutter/packages/pull/6403](https:/flutter/packages/pull/6403), where it was decided that we should not replace the `ProrationMode` with `ReplacementMode`, but instead only deprecate `ProrationMode`. The reason for a new PR is also that `in_app_purchase_android` version `0.3.3` changed internal platform communication to Pigeon, which meant I had to make major changes to my original PR.

*List which issues are fixed by this PR. You must list at least one issue.*
flutter/flutter#128957
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App p: in_app_purchase platform-android
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants