renanzdm / ping_discover_network_forked

BSD 3-Clause "New" or "Revised" License
2 stars 7 forks source link

Support multiple platforms #1

Open guyluz11 opened 3 years ago

guyluz11 commented 3 years ago

Currently we are getting 10 less points in pub.dev for missing support for multiple platforms

image

This is caused because dart:io is missing support for web and more for that we can use universal_io that is

A cross-platform dart:io that works in all platforms (browsers, mobile, desktop, and server-side).

Would you like me to test it on the browser and implement it? All tests are passing with universal_io.

renanzdm commented 3 years ago

Of course any contribution is welcome, I'm a little busy right now and I can't. Your help will be welcome

renanzdm commented 3 years ago

@guyluz11 Did you make the changes you mentioned?

guyluz11 commented 3 years ago

I have tested a small part and found out that this is will not work by simply adding universal_io, the program will compile but this is not enough for it to work.

I think I have asked about this in another package and they gave me pretty good explanation why this can't be done for the browser, something about chrome does not support this type of call.

Maybe we can add support for native dart, that could be useful too, but currently I don't have time to do that.