-
Notifications
You must be signed in to change notification settings - Fork 9.7k
[connectivity] announce 1.0, deprecate wifi APIs #3183
Conversation
amirh
left a comment
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
| expect(result, isNotNull); | ||
| switch (result) { | ||
| case ConnectivityResult.wifi: | ||
|
|
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.
extra line?
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.
done
ditman
left a comment
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! it needs a pass of dartfmt -w . however.
| /// 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') |
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.
Why not a link to pub.dev? It'd be much shorter!
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.
wifi_info_flutter is not published yet :(
Description
Bump the version to 1.0.0, announce deprecations to all wifi related APIs