Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[FrameworkBundle] Messenger requires Serializer#26905
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
sroze commentedApr 12, 2018
It actuallyisn't :). |
chalasr commentedApr 12, 2018
The Messenger component is usable without the Serializer one, it's a soft dependency only. See#26649 which fixed an error when the serializer wasn't installed, should be all good now. |
sroze commentedApr 12, 2018
Though, we should work at making the error message clearer. I've already discussed this and I believe the best way is to have the |
sroze commentedApr 12, 2018
Thanks for the PR but as per our comments, I'm going to close this one. |
boite commentedApr 12, 2018
ah, I see. I was getting this though:- |
sroze commentedApr 12, 2018
It will be more explicit with#26908 :) |
The new Messenger component's integration with FrameworkBundle (c9cfda9) makes the latterdepend on the Serializer component. This PR moves the dependency from
require-devtorequirein composer.json.