-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[DependencyInjection][Messenger] Add auto-registration for BatchHandlerInterface
#44490
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
Conversation
GaryPEGEOT
commented
Dec 7, 2021
Q | A |
---|---|
Branch? | 5.4 |
Bug fix? | yes |
New feature? | no |
Deprecations? | no |
Tickets | |
License | MIT |
Doc PR | - |
|
Thank you @GaryPEGEOT. |
@GaryPEGEOT hi there! I know it's been a long time, but i'm curious why this change was introduced? I've created an issue #59912, but I'm wondering if I'm missing something.. Do you remember why would we need to autoregister services with BatchIndexHandlerInterface with Thanks! |
BatchHandlerInterface
@oneNevan it was so a class implementing this interface would be auto-configured with the correct tag. I might be wrong, but I think it was before |
@GaryPEGEOT Yes, you are probably right! Now I recall we previously had MessageHandlerInterface for regular handlers which was deprecated with introduction AsMessageHandler attribute and removed in symfony 7.0 Thank you! |