-
-
Notifications
You must be signed in to change notification settings - Fork 940
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(connectivity_plus): vpn support for android and linux #1174
feat(connectivity_plus): vpn support for android and linux #1174
Conversation
a82472b
to
e3bbce5
Compare
Hey! Due to changes in our contributor guidelines, all PRs should now:
We would ask you kindly to update the PR following these changes. Thanks! |
done ✅
ya i got to know. i haven't modified these files, instead have added the checkbox stating i "haven't" done that. |
@@ -56,12 +60,14 @@ private String getNetworkTypeLegacy() { | |||
int type = info.getType(); | |||
switch (type) { | |||
case ConnectivityManager.TYPE_BLUETOOTH: | |||
return "bluetooth"; | |||
return CONNECTIVITY_BLUETOOTH; |
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.
Thanks for the improvement!
completion(equals(ConnectivityResult.vpn)), | ||
); | ||
}); | ||
|
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.
thanks for including tests!
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.
Thanks for another great contribution!
Description
adds support for vpn as
ConnectivityResult
to android and linux.for android - added it as there is a parameter for it, but haven't tested it on device as i don't have a VPN.
for linux - again, added it (taking the word of the issue author) but can't test it on device.
for macos / ios - i suppose they don't support knowing if a vpn is active as i wasn't able to find any doc supporting it. the APIs currently use don't have such a parameter.
for windows, web - tried searching for something, but not an expert so will leave it to them / a feature request to come up.
Related Issues
Fixes #1166
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.
pubspec.yaml
andCHANGELOG.md
.///
).flutter analyze
) does not report any problems on my PR. (melos analyze
fails locally but not related to my PR).Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?