- Issue created by @lussoluca
- 🇦🇺Australia dpi Perth, Australia
Looks like interface is required now.
-
dpi →
committed efbaf9cc on 1.x authored by
lussoluca →
Issue #3472735 by lussoluca: Queue factories must implement...
-
dpi →
committed efbaf9cc on 1.x authored by
lussoluca →
- Status changed to Fixed
3 months ago 12:47pm 8 September 2024 - 🇦🇺Australia dpi Perth, Australia
Merged, thanks.
Also working on test/CI fixes and a few service changes.
If you're working with Drupal 11 already, I'd appreciate if you could check if things works as expected, in the next tagged release.
- 🇦🇺Australia dpi Perth, Australia
Also from 📌 Convert QueueFactory to use a service locator Fixed , it seems the new locator only contains one version of the service, so not the alias of `SmLegacyQueueFactory`.
Since the README advertises:
$settings['queue_default'] = \Drupal\sm\QueueInterceptor\SmLegacyQueueFactory::class;
Im switching this class to be the canonical service name, so the old `sm.queue_interceptor.queue_item_factory` is now an alias.
Im not sure why the locator for cores `queue` service doesnt get both, it probably should.
-
dpi →
committed efbaf9cc on gitlab-checkin authored by
lussoluca →
Issue #3472735 by lussoluca: Queue factories must implement...
-
dpi →
committed efbaf9cc on gitlab-checkin authored by
lussoluca →
Automatically closed - issue fixed for 2 weeks with no activity.