Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

[in_app_purchase] Readme updates #1798

Merged
merged 4 commits into from
Jul 8, 2019

Conversation

moritzmorgenroth
Copy link
Contributor

@moritzmorgenroth moritzmorgenroth commented Jul 4, 2019

I made the following fixes to the code samples in the documentation, as they contained some syntax errors / errors:

  • Remove explicit typing in _subscription callback
  • Remove const modifier for _kIds list
  • Fix not found ids check typos
  • Migrate loop iterator from : to in

Description

In noticed a few glitches in the docs for the in_app_purchase plugin when integrating with an up-to-date flutter app.

Related Issues

Checklist

Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
  • All existing and new tests are passing.
  • I updated/added relevant documentation (doc comments with ///).
  • The analyzer (flutter analyze) does not report any problems on my PR.
  • I read and followed the Flutter Style Guide.
  • The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
  • I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
  • I updated CHANGELOG.md to add a description of the change.
  • I signed the CLA.
  • I am willing to follow-up on review comments in a timely manner.

Breaking Change

Does your PR require plugin users to manually update their apps to accommodate your change?

  • Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
  • No, this is not a breaking change.

CHANGELOG
* Remove explicit typing in `_subscription` callback
* Remove `const` modifier for `_kIds` list
* Fix not found ids check typos
* Migrate loop iterator from `:` to `in`
@moritzmorgenroth moritzmorgenroth changed the title [in_app_purchase] Documentation updates [in_app_purchase] Readme updates Jul 4, 2019
@cyanglaz cyanglaz added documentation submit queue The Flutter team is in the process of landing this PR. labels Jul 4, 2019
Copy link
Contributor

@cyanglaz cyanglaz left a comment

Choose a reason for hiding this comment

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

LGTM! cc @mklim as the original README author.

@collinjackson
Copy link
Contributor

LGTM as well, I fixed one more typo and will go ahead and merge this as it seems like a low-risk documentation change.

@collinjackson collinjackson merged commit 5e48cd8 into flutter:master Jul 8, 2019
mithun-mondal pushed a commit to bKash-developer/archived_plugins that referenced this pull request Aug 6, 2019
* Documentation updates

CHANGELOG
* Remove explicit typing in `_subscription` callback
* Remove `const` modifier for `_kIds` list
* Fix not found ids check typos
* Migrate loop iterator from `:` to `in`
* Fix typo from flutter#1798
julianscheel pushed a commit to jusst-engineering/plugins that referenced this pull request Mar 11, 2020
* Added unit test for the onAuthStateChanged stream (from JS).
* Migrate tests from jsify to package:js
* Update version and CHANGELOG

Fixes firebase/flutterfire#1685
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes documentation submit queue The Flutter team is in the process of landing this PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants