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

[shared_preferences] Endorse web plugin. #2390

Merged
merged 4 commits into from
Dec 11, 2019
Merged

[shared_preferences] Endorse web plugin. #2390

merged 4 commits into from
Dec 11, 2019

Conversation

harryterkelsen
Copy link
Contributor

Description

Endorse the web plugin in shared_preferences.

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.

Copy link
Member

@ditman ditman left a comment

Choose a reason for hiding this comment

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

LGTM, except some questions about shared_preferences_web version number.

# validation, so we set a ^ constraint.
# TODO(amirh): Revisit this (either update this part in the design or the pub tool).
# https://github.com/flutter/flutter/issues/46264
shared_preferences_web: ^0.1.1
Copy link
Member

Choose a reason for hiding this comment

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

I'd bump shared_preferences_web to 0.1.2, so you can do ^0.1.2 here.

(Because there was a problem with ^0.1.1+1 right?)

Copy link
Contributor

@amirh amirh left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Member

@ditman ditman left a comment

Choose a reason for hiding this comment

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

Thanks for adding the web example!

@harryterkelsen harryterkelsen merged commit fc2ea9f into flutter:master Dec 11, 2019
eugenejeonme added a commit to COMANDKEYINC/flutter_plugins that referenced this pull request Dec 11, 2019
This reverts commit fc2ea9f.

# Conflicts:
#	packages/shared_preferences/shared_preferences/CHANGELOG.md
#	packages/shared_preferences/shared_preferences/pubspec.yaml
FlutterSu pushed a commit to FlutterSu/flutter-plugins that referenced this pull request Nov 20, 2020
* Make shared_preferences example work on web

* [shared_preferences] Endorse web plugin

* format pubspec
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
4 participants