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

support for dynamically subscribing to new topics after start #58

Closed
wants to merge 1 commit into from

Conversation

dams
Copy link
Contributor

@dams dams commented Jan 16, 2018

code is straightforward. I'm using this to pilot what my application is subscribed to. Feedback welcome

@dams
Copy link
Contributor Author

dams commented Jan 19, 2018

see the counterpart: #59

@dams
Copy link
Contributor Author

dams commented Apr 3, 2018

ping ?

@dams
Copy link
Contributor Author

dams commented Jun 1, 2018

Any progress ?

@objectuser objectuser closed this in a4a396c Jun 4, 2018
@objectuser
Copy link
Contributor

@dams I merged this in. I refactored a few things. LMK if anything isn't what you'd expect.

I'm going to work the unsubscribe next.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants