- πΊπΈUnited States xjm
Removing a thing from core requires signoff from all the committer roles, as well as an opportunity for the subsystem maintainer to provide feedback if there is one. (There is none here.)
The committer team recently reviewed our scoring exercise on all core modules. There is a reasonably strong consensus that Action is neither a foundational capability nor strategically valuable to the product roadmap, so it should be removed. I am marking RTBC, but leaving the tags on to give committers a chance to confirm that they agree.
- πΊπΈUnited States xjm
Myself, @catch, @quietone, and @longwave all agree on moving Action to contrib.
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
I'm happy moving this if we're talking about 'Action UI' module i.e. keeping the system parts of action APIs that drive things like bulk actions at admin/content for example.
- π¬π§United Kingdom catch
Just to confirm #8, the actual actions are provided by system module and are available whether this module is installed or not, this is just the UI for creating 'advanced actions'.
- π©πͺGermany jurgenhaas Gottmadingen
Thanks @catch for clarifying this. So the ActionPluginManager will stay in core even if the Action UI module moved to contrib, right? That would be ideal from our ECA module point of view, because that's what we rely on. And core is our sole dependency, we would certainly want to keep that if possible.
- Status changed to Fixed
almost 2 years ago 11:28am 20 February 2023 - ππΊHungary GΓ‘bor Hojtsy Hungary
Clarifying the title. I agree the Action UI module included in core is very basic and does not really allow the capabilities of actions to shine. (Just look at ECA!!) Indeed its used rarely, not maintained and does not have a contrib ecosystem that extends this UI in particular. Signing off as one of the core product managers.
@jurgenhaas: https://api.drupal.org/api/drupal/core%21core.services.yml/service/plugi... is the service I think you mean? It is in core.services already not in the UI module :)
- π«π·France andypost
@jurgenhaas in terms of ECA module, there's related issues
- β¨ Use Context system for actions Needs work
- π Rename action module to action_ui Closed: outdated - π©πͺGermany jurgenhaas Gottmadingen
Thanks @GΓ‘bor and @andypost this is much clearer now. And yes, the context system is on our radar for ECA.
- πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
Do we need a child issue for the actual deprecation patch?
Do we want a "action system" in the Drupal project issue queue? Right now it's really easy to classify something accidentally as "action.module" instead of "system.module" - π«π·France andypost
It needs new meta for that, as there's separate issue to decouple test groups
Automatically closed - issue fixed for 2 weeks with no activity.