Skip to content
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

Web Platform Support #25

Closed
wants to merge 2 commits into from
Closed

Web Platform Support #25

wants to merge 2 commits into from

Conversation

OutdatedGuy
Copy link

@OutdatedGuy OutdatedGuy commented Jun 20, 2022

Status

READY

Breaking Changes

YES

Description

Changes

Problem

  • Socket.connect is not implemented for web platform in universal_io package

Solution

  • Using
    html.window.navigator.onLine
    to get connection status for web

Type of Change

  • ✨ New feature (non-breaking change which adds functionality)
  • 🛠️ Bug fix (non-breaking change which fixes an issue)
  • ❌ Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 Code refactor
  • ✅ Build configuration change
  • 📝 Documentation
  • 🗑️ Chore

- Allows use of InternetAddress on web platform

resolves #15 #1
- using flutter to check for web platform and use navigator method
- using universal html to get connection status from window navigator
@kjawadDeveloper
Copy link

Was there any reason that it was closed?

@OutdatedGuy
Copy link
Author

Was there any reason that it was closed?

The PR was only able to detect if the device is connected to any network on the web. Not the actual internet connection status.

It's currently not fixable with using Socket connections. Hence I created my own package to check with http connection to support the web platform as well.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Web Unsupported operation: InternetAddress No working (Tested on web)
2 participants