Skip to content
This repository has been archived by the owner on Dec 12, 2021. It is now read-only.

Only subscribe once to a channel #70

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nfm
Copy link

@nfm nfm commented Jan 26, 2013

No description provided.

@ryanb
Copy link
Owner

ryanb commented Feb 22, 2013

Thank you for the pull request. Sorry for the delay in checking this out. Any chance we could get a test for this change? I'd be happy to merge then. Thanks!

@ryanb
Copy link
Owner

ryanb commented Feb 22, 2013

BTW, I merged in pull request #60 which conflicts with this. It may also resolve this issue. If it isn't resolved, please update to master and add specs. Thanks.

@nasreddineskandrani
Copy link

What is the status of this request?
I still have the problem of multiple subscribe in the master of private_pub.

the fix proposed by vollnhals works for me. You can merge!
this fork fix it until you merge this to master.
https://github.com/nasreddineskandrani/private_pub

cheers

ryleto added a commit to ryleto/private_pub that referenced this pull request Feb 16, 2016
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.

4 participants