-
Notifications
You must be signed in to change notification settings - Fork 55
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
Consider moving away from server sent events as not supported in IE - SignalR? #1
Comments
We could fork the solution and try SignalR. The good thing about SSE is that it's done over HTTP (i.e. doesn't require another protocol) and is very lightweight. |
+1 for this. IE support would be great. |
I agree with @sbarski, SSE is a standard, it's only a shame that IE does not implement it. |
It would probably mean creating two server side broadcasting points but I wonder if it would be possible to implement an EventSource object that would provide polling/websockets if SSE were not available & allow reuse of existing shopify code. |
I think we should maintain SSE as our standard. I think we should keep this (canonical) version of dashing.net compatible with the original dashing and with the other implementations (in python and perl) that are currently being written. For IE support we have a couple of options:
There might be other polyfills we could use or we could roll our own. This could be my weekend project but if anyone want to give it a shot - please feel free :) |
I think the polyfill option is be the best one. It respects the open-close principle (sort of). We don't change anything, we just extend when needed. It's even an interesting addition to the original dashing project, I'm sure they would be happy to merge a PR that make them support IE (hum hum). |
Did this ever get any further? The demo doesnt currently seem to work in IE though. |
How can we make it to support IE? Actually IE still has big share in browsers. We have to support it before we use this. |
No description provided.
The text was updated successfully, but these errors were encountered: