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

Jump to bottom

[www+cli] Add websocket route + notification subscription system #547

Closed
marcopeereboom opened this issue Oct 18, 2018 · 0 comments · Fixed by #668
Closed

[www+cli] Add websocket route + notification subscription system #547

marcopeereboom opened this issue Oct 18, 2018 · 0 comments · Fixed by #668
Labels
enhancement The issue enhances an existing feature.

Comments

@marcopeereboom
Copy link
Member

In order to complete #372 we need to implement a websockets route. We are going to use gorilla websockets (https://github.com/gorilla/websocket) because they can be upgraded on certain paths. This means we'll move certain commands to become websocket based.

When this is implemented a generic notification framework needs to be added that clients can subscribe to.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement The issue enhances an existing feature.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add websockets to provide server notifications. marcopeereboom/politeia
1 participant
@marcopeereboom