[Messenger] Add a simple serializer #28400
Conversation
31a0ba0 to
4b1b74f
Compare
|
I haven't used the Messenger component yet so I'm probably asking something stupid but --> could we make a radical simplification for this "SimpleSerializer" and just use |
|
random thought, but what about putting a |
|
It's a bit more involving than that, but having the serializer as a dependency does not add any complexity, so I don't think we need to make it optional. SimpleSerializer is only needed when you are using the component standalone where you would basically have to do what I've done here. So, it only to get you started faster. |
|
@ro0NL That's my thoughts as well :) |
src/Symfony/Component/Messenger/Transport/Serialization/SimpleSerializer.php
Outdated
Show resolved
Hide resolved
43696a2 to
1c5890a
Compare
1c5890a to
f27c15a
Compare
This PR was merged into the 4.2-dev branch. Discussion ---------- [Messenger] Add a simple serializer | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | n/a | License | MIT | Doc PR | #... When using the Messenger component without Symfony full stack, it helps to use a simple Serializer configured with the bare minimum (this bare minimum is up to the discussion). Commits ------- f27c15a [Messenger] added a simple serializer
When using the Messenger component without Symfony full stack, it helps to use a simple Serializer configured with the bare minimum (this bare minimum is up to the discussion).