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
[Messenger] Document middleware factories #9774
[Messenger] Document middleware factories #9774
Conversation
17dae2f
to
4deadc1
Compare
messenger.rst
Outdated
@@ -273,6 +273,114 @@ you can disable them like this: | |||
messenger.bus.default: | |||
default_middleware: false | |||
|
|||
Using middleware factories |
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.
Using Middleware Factories
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.
Fixed thanks
4deadc1
to
2880027
Compare
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.
Thanks a lot @javiereguiluz for improving this PR ❤️
messenger.rst
Outdated
Using Middleware Factories | ||
~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
|
||
Sometimes middleware are configurable using factories. There are two types of |
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.
Turned like this, it seems to me the sentence is saying that there are two different ways to achieve configurable middleware, while it's actually two required services definitions. One registering the factory class as a service. The second one allowing to create a middleware instance using the first factory.
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.
I'm sorry for having changed the meaning of the docs. I've reworded it again to follow your explanation. Cheers!
Thanks Maxime. |
…iereguiluz) This PR was merged into the 4.1 branch. Discussion ---------- [Messenger] Document middleware factories <!-- If your pull request fixes a BUG, use the oldest maintained branch that contains the bug (see https://symfony.com/roadmap for the list of maintained branches). If your pull request documents a NEW FEATURE, use the same Symfony branch where the feature was introduced (and `master` for features of unreleased versions). --> Refs: - Symfony PR: symfony/symfony#27128 - DoctrineBundle PR: doctrine/DoctrineBundle#817 Commits ------- c5ef7c8 Reowrd to restore the original meaning a87f21b Minor rewords 2880027 [Messenger] Document middleware factories
Refs: