-
Notifications
You must be signed in to change notification settings - Fork 9.8k
[in_app-purchase] Updated the ReadMe to remove the deprecated InAppPurchaseAndroidPlatformAddition.enablePendingPurchases
method
#4597
Conversation
@mvanbeusekom Hello, I am wondering if you could review this and merge it in so that the new plugin user won't get confused by the ReadMe 😸 |
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.
Looks good, however you should also update the CHANGELOG.md mentioning the change to the README. Something like:
## 2.0.1
* Removes the instructions on initializing the plugin since this functionality is deprecated.
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, however there seems to be an issue with the CLA. It looks like you pushed commit 8cf9680 with a different email address.
Oh that's why haha. Thank you! will do |
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, thanks!
… `InAppPurchaseAndroidPlatformAddition.enablePendingPurchases ` method (flutter/plugins#4597)
… `InAppPurchaseAndroidPlatformAddition.enablePendingPurchases ` method (flutter/plugins#4597)
… `InAppPurchaseAndroidPlatformAddition.enablePendingPurchases ` method (flutter/plugins#4597)
… `InAppPurchaseAndroidPlatformAddition.enablePendingPurchases ` method (flutter/plugins#4597)
…urchaseAndroidPlatformAddition.enablePendingPurchases ` method (flutter#4597) This will resolve flutter/flutter#93837 & this work is just the follow-up from flutter#4527 PR. We need to update the documentation in the README.md of the in_app_purchase package to remove the instruction of initialization with the deprecated `InAppPurchaseAndroidPlatformAddition.enablePendingPurchases` method. Fixes flutter/flutter#94977
… `InAppPurchaseAndroidPlatformAddition.enablePendingPurchases ` method (flutter/plugins#4597)
…urchaseAndroidPlatformAddition.enablePendingPurchases ` method (flutter#4597) This will resolve flutter/flutter#93837 & this work is just the follow-up from flutter#4527 PR. We need to update the documentation in the README.md of the in_app_purchase package to remove the instruction of initialization with the deprecated `InAppPurchaseAndroidPlatformAddition.enablePendingPurchases` method. Fixes flutter/flutter#94977
This will resolve flutter/flutter#93837 & this work is just the follow-up from #4527 PR.
We need to update the documentation in the README.md of the in_app_purchase package to remove the instruction of initialization with the deprecated
InAppPurchaseAndroidPlatformAddition.enablePendingPurchases
method.Fixes flutter/flutter#94977
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.///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.