- Issue created by @JeroenT
- @jeroent opened merge request.
- @jeroent opened merge request.
- Status changed to Needs review
over 1 year ago 8:16am 28 March 2023 - π¨πSwitzerland berdir Switzerland
I think it's more common to do the opposite and register the service in the service provider instead of removing it? Both has it's advantages, I guess having it defined increases its discoverability, but I don't think this needs to be discovered as you don't have to use it.
- Status changed to Needs work
over 1 year ago 5:00pm 28 March 2023 - πΊπΈUnited States papagrande US West Coast
Based on @berder's comment, setting this back to NW.
-
Berdir β
committed 7059ad3e on 8.x-1.x authored by
JeroenT β
Issue #3350803 by JeroenT: Error : Class "Drupal\layout_builder\Plugin\...
-
Berdir β
committed 7059ad3e on 8.x-1.x authored by
JeroenT β
- Status changed to Fixed
over 1 year ago 6:43pm 28 March 2023 - π¨πSwitzerland berdir Switzerland
It's a fairly minor thing, I had a minute and commented in case someone has time to improve it. That said, this is much better than not working at all, so fixing this getting a release out is of higher prio than nitpicking.
There's no test coverage, but replicate_ui is actually failing right now. I was trying to get the release out and didn't properly check that, sorry.
- πΊπΈUnited States papagrande US West Coast
I confirmed that all is working again after updating. Thanks for the quick turnaround, @Berdir & @JeroenT.
Automatically closed - issue fixed for 2 weeks with no activity.