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

Update config_reference.md #326

Merged
merged 1 commit into from Jan 12, 2018
Merged

Update config_reference.md #326

merged 1 commit into from Jan 12, 2018

Conversation

errogaht
Copy link
Contributor

it is working for me with this configuration. I use "enqueue/enqueue-bundle": 0.8.13.
with your version when I run bin/console enqueue:consume default --setup-broker i receiving Processor was not found. processorName: "enqueue.client.router_processor"

it is working for me with this configuration. I use "enqueue/enqueue-bundle": 0.8.13.
with your version when I run bin/console enqueue:consume default --setup-broker i receiving Processor was not found. processorName: "enqueue.client.router_processor"
@makasim
Copy link
Member

makasim commented Jan 12, 2018

It works for me, it works in tests and it is not clear how changing the value to the class is solves an issue. enqueue.client.router_processor is an alias to the class.

But most importantly the doc contains an output of ./bin/console config:dump-reference enqueue and should not be edited manually.

@makasim makasim closed this Jan 12, 2018
@errogaht
Copy link
Contributor Author

hmm. why it's not working for me? I have symfony 2.8. and when client: ~ all work fine, but when I put parameters block with client from documentation got "Processor was not found. processorName: "enqueue.client.router_processor"" then if I change router_processor: to Enqueue\Client\RouterProcessor all work fine. now all work fine for me, maybe it can useful for you guys, I don't need any help with this because I fix it

@makasim
Copy link
Member

makasim commented Jan 12, 2018

the doc is out of sync with the code, your fix is correct.

@makasim makasim reopened this Jan 12, 2018
@makasim makasim merged commit 8b6ca1d into php-enqueue:master Jan 12, 2018
@makasim
Copy link
Member

makasim commented Jan 12, 2018

Thank you

ASKozienko pushed a commit that referenced this pull request Nov 2, 2018
Update config_reference.md
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.

None yet

2 participants