Problem/Motivation

Track the steps needed to remove extension action from core to contrib. See Remove a core module and move it to a contributed project of the deprecation policy.

The removal of extension action was approved in #3266458: [Policy] Deprecate Action (UI) module in D10 and move to contrib in D11.

Remaining tasks

  1. Create a section on Deprecated and obsolete modules and themes to provide recommendations for sites using extension action machine name. The recommendations are to include instructions for sites using the extension and for contributed projects that depend on then extension.
  2. Set lifecycle to deprecated.
  3. Set lifecycle_link to the section added above, https://www.drupal.org/node/3223395#s-action.
  4. Add @group legacy to the tests in the extension.
  5. The change record for this issue includes a link to the doc page.
  6. Do a thorough search of core for any remaining references to the extension. If references are found, outside of the extension, then creates issues to remove the references.

Release notes snippet

Issue fork drupal-3369912

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

quietone created an issue. See original summary.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Issue summary: View changes
Status: Postponed » Active
Related issues: +#3344683: Unreserve or create "action" module in contrib

andypost opened merge request !6860

andypost’s picture

andypost
he/him
Primary language Russian
commented
Title: Final steps to deprecate Actions module » Final steps to deprecate Actions UI (action) module
Status: Active » Needs work

Filed Draft CR https://www.drupal.org/node/3425074 and started MR to deprecate

now it needs new section at https://www.drupal.org/node/3223395#s-upcoming-deprecations

andypost’s picture

andypost
he/him
Primary language Russian
commented

Maybe file separate issue for changes from https://git.drupalcode.org/project/drupal/-/merge_requests/6860/diffs?co... commit?

andypost’s picture

andypost
he/him
Primary language Russian
commented

Initial release created but not installable https://www.drupal.org/project/action

andypost’s picture

andypost
he/him
Primary language Russian
commented

now module installs and pass tests

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs work » Needs review

As tests are green after rebase, I think it's the last step!

andypost’s picture

andypost
he/him
Primary language Russian
commented

Docs updated https://www.drupal.org/node/3223395#s-action

andypost’s picture

andypost
he/him
Primary language Russian
commented
Issue summary: View changes
andypost’s picture

andypost
he/him
Primary language Russian
commented
Issue summary: View changes
andypost’s picture

andypost
he/him
Primary language Russian
commented
Issue summary: View changes
spokje’s picture

spokje
Primary language Dutch
commented
Status: Needs review » Reviewed & tested by the community

Thanks @andypost!

Made some minimal changes to the CR, most notably the link to the recommendations.

6. Do a thorough search of core for any remaining references to the extension. If references are found, outside of the extension, then creates issues to remove the references.

Nice job on finding the two stragglers, the above from the IS would suggest making separate issue(s) for them.
Looking at how minimal the changes are, I think it's not worth the admin-overhead, so I'm marking this RTBC.

  • catch committed 24cb8bc4 on 10.3.x 
    Issue #3369912 by andypost, quietone, Spokje: Final steps to deprecate...

  • catch committed c703f7ba on 11.x 
    Issue #3369912 by andypost, quietone, Spokje: Final steps to deprecate...
catch’s picture

catch
he/him
Primary language English
commented
Version: 11.x-dev » 10.3.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 11.x and cherry-picked to 10.3.x, thanks!

catch closed merge request !6860

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented

There are still a couple of TBA things in the issue summary, should we update those to be correct?

catch’s picture

catch
he/him
Primary language English
commented
Issue summary: View changes

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.