-
-
Notifications
You must be signed in to change notification settings - Fork 937
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
fix(connectivity_plus): Add connectivity_plus_web export #1278
Conversation
In my case, |
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, although I didn't need this fix for me to work, I guess some users may need this depending on the SDK version they are running.
This issue was before the sdk change 🤷🏽♂️ |
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.
I hope that this fix will be the last one for this move to 2 package structure 😄
Well, the Android test is still failing 🙈 |
no worry about this, it is because the API 21 integration test fails, but it works on more recent Android versions ;) |
Can we fix the test? Another pub.dev/GitHub release today? 🙂 |
Description
This PR adds a missing export.
Related Issues
#1227 (comment)
Checklist
CHANGELOG.md
nor thepubspec.yaml
files.flutter analyze
) does not report any problems on my PR.Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?
!
in the title as explained in Conventional Commits).