LiveQuery needs refactoring

[rostopira]

Current implementation spawns new separate websocket connection for every subscription
This is huge overkill and will likely cause additional load on server

[arvanus]

@rostopira are LiveQuery working for you?
See #198

[rostopira]

Yes, it works, because I'm using release branch, read last comment on issue
It also works on Flutter Web

[phillwiggins]

Hey @rostopira

Can you look at the Web Live Query stuff you wrote. It seems to be listing invalid libraries and has dragged our Flutter library score on Pub Dev to a really low level. It looks as though something is missing? It ideally needs a hotfix pretty promptly.

[rostopira]

The only thing that it uses is dart:html, and it's valid library. But Dart Analyzer doesn't think so, see: flutter/flutter#35588

[rostopira]

Actually, using import 'dart:html' as HTML; make dart analyzer scream on this line only, so I will provide PR with that
UPD: this should make analyzer shut up

// ignore: uri_does_not_exist
import 'dart:html' as HTML;

[rostopira]

@phillwiggins PR is ready to merge

[phillwiggins]

Greatly appreciated!

…

On Fri, Aug 23, 2019, 17:12 Dima Rostopira ***@***.***> wrote: Actually, using import 'dart:html' as HTML; make dart analyzer scream on this line only, so I will provide PR with that — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#217>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AB4CPXVRB5KD6HDA6VUP3XTQGAD7LANCNFSM4H7RJS6Q> .