-
Notifications
You must be signed in to change notification settings - Fork 436
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
fix fsdriver router topic name #34
Conversation
4334b16
to
c10f0de
Compare
pkg/enqueue/Client/Config.php
Outdated
* | ||
* @return string | ||
*/ | ||
private function getQueueName($name) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please rename it to getDestinationName
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
c10f0de
to
7a5d2f0
Compare
I have just realized that the router topic must not contain an app it. The router topic is a shared one. Several applications may send messages to it and hence talk to each other. It is a ground part of message bus support. And since message bus is not supported by |
no problem, i'll take a look. |
7a5d2f0
to
d91cc99
Compare
@bendavies yeap that looks good. |
@makasim i'll add a test to cover this case. |
d91cc99
to
93b18b2
Compare
@makasim done |
Thanks! |
As discussed in gitter.