-
Notifications
You must be signed in to change notification settings - Fork 595
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
Add support for Server-Sent Events by including Akka SSE #669
Comments
Akka SSE: https://github.com/hseeberger/akka-sse |
I have just released Akka SSE 2.0.0. The idea is that it could be moved in as is (except for shuffling package names and adding non-Scaladoc documentation), so please take a look at the design and implementation. |
Great, thanks a lot Heiko. I'll get to it next week I think, attempting to stay offline for a few days now :) |
Hope to get to it this week, we're still halfed with much of the team on holidays. Sorry for the delay. |
Any progress here? |
Could you make a PR with it? Would make reviewing much easier hm.
…On Jan 14, 2017 16:00, "Heiko Seeberger" ***@***.***> wrote:
Any progress here?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#669 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAHYk0py3720eLUkzHftUgFWmk-bEqUNks5rSOMQgaJpZM4LSQqi>
.
|
Sure, I could create a PR. But that's a lot of work and therefore I'd like to see a general thumbs up to the idea of including SSE first. |
As you know since a long time, there is a thumbs up ;-) |
Sounds great! Will make a PR as soon as I can find some time. |
(didn't check the thumbs-up, now there are four ;-)) |
Excellent, thanks 👍 |
@hseeberger Any chance you could submit that PR soon? Looking to use this for a project and I admit it would be easiest if this was bundled directly with akka-http. Also makes sense as they are already supporting WebSockets :) . |
@OverlyExcessive I'm too busy, but using akka-sse should be a no-brainer. |
Please discuss.
The text was updated successfully, but these errors were encountered: