Skip to content
This repository has been archived by the owner on Feb 11, 2020. It is now read-only.

Server.subscribe #411

Closed
wants to merge 32 commits into from
Closed

Server.subscribe #411

wants to merge 32 commits into from

Conversation

behrad
Copy link
Contributor

@behrad behrad commented Feb 6, 2016

provide a subscribe method using ascoltatori backend. Closes #408

@mcollina
Copy link
Collaborator

mcollina commented Feb 6, 2016

I would just throw in a little unit test for the new API.

using newPacket instead of packet for publish
@mcollina
Copy link
Collaborator

Any update on this?

@behrad
Copy link
Contributor Author

behrad commented Feb 22, 2016

I was busy @mcollina , sorry for being late, will update ASAP

@mcollina
Copy link
Collaborator

mcollina commented May 2, 2016

Can I get a couple of tests for this, so I add it to #464 ?

@behrad
Copy link
Contributor Author

behrad commented May 12, 2016

this PR seemed abit old, I created a new one based on your master #472 However there's also a conflict! I can't say why!?

@mcollina
Copy link
Collaborator

It's not based on latest master.

@mcollina mcollina closed this May 12, 2016
@behrad behrad deleted the server.subscribe branch May 12, 2016 11:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants