Thanks to visit codestin.com
Credit goes to github.com

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

[connectivity] announce 1.0, deprecate wifi APIs #3183

Merged
merged 8 commits into from
Oct 22, 2020

Conversation

cyanglaz
Copy link
Contributor

Description

Bump the version to 1.0.0, announce deprecations to all wifi related APIs

@google-cla google-cla bot added the cla: yes label Oct 21, 2020
@cyanglaz cyanglaz requested a review from amirh October 21, 2020 19:27
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

@@ -22,6 +22,7 @@ void main() {
expect(result, isNotNull);
switch (result) {
case ConnectivityResult.wifi:

Copy link
Contributor

Choose a reason for hiding this comment

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

extra line?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

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! it needs a pass of dartfmt -w . however.

@@ -63,11 +65,15 @@ class Connectivity {
///
/// From Android 8.0 onwards the GPS must be ON (high accuracy)
/// in order to be able to obtain the BSSID.
@Deprecated(
'This method is deprecated. Please use wifi_info_flutter instead. See https://github.com/flutter/plugins/blob/master/packages/wifi_info_flutter/wifi_info_flutter/README.md')
Copy link
Member

Choose a reason for hiding this comment

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

Why not a link to pub.dev? It'd be much shorter!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

wifi_info_flutter is not published yet :(

@cyanglaz cyanglaz merged commit 30ee306 into flutter:master Oct 22, 2020
@cyanglaz cyanglaz deleted the connectivity_1.0 branch October 22, 2020 02:20
FlutterSu pushed a commit to FlutterSu/flutter-plugins that referenced this pull request Nov 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants