-
Notifications
You must be signed in to change notification settings - Fork 9.8k
[connectivity] announce 1.0, deprecate wifi APIs #3183
Conversation
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
@@ -22,6 +22,7 @@ void main() { | |||
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
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.
@@ -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') |
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