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

Fixed bind to allow not specifying callback #1

Merged
merged 2 commits into from
Feb 12, 2014
Merged

Fixed bind to allow not specifying callback #1

merged 2 commits into from
Feb 12, 2014

Conversation

grantila
Copy link
Contributor

When binding an exchange to a queue which already has messages in it, one might want to be able to not subscribe to the queue with default options, e.g. to be able to use 'ack'.
This fix enables you to only bind, so that you can subscribe later at any stage.

When binding an exchange to a queue which already has messages in it, one might want to be able to not subscribe to the queue with default options, e.g. to be able to use 'ack'.
This fix enables you to only bind, so that you can subscribe later at any stage.
@algesten
Copy link
Contributor

We can incorporate this, however it's a bit superfluous since you can declare the queue first bind it second – the specific bind on the root object is specifically a "shortcut" (as noted by the documentation).

Having said that. Update the README.md documenting this feature, and we'll merge the pull.

Cheers!
M

@grantila
Copy link
Contributor Author

You're right about this being a bit superfluous, but it can make the user code slightly nicer.

Thanks for the merge!

algesten added a commit that referenced this pull request Feb 12, 2014
Fixed bind to allow not specifying callback
@algesten algesten merged commit 1dcf3cf into ttab:master Feb 12, 2014
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