Problem/Motivation

Remaining tasks

  1. The change record for this issue should include a link to recommendations page, https://www.drupal.org/node/3223395#s-settings-tray. (For example, the CR for removing HAL)
  2. Tag this issue 'Needs release note.'
  3. Remove the extension ;-).
  4. Update update path tests as needed. See Learn how to write an automated update test.
  5. Remove references from core/phpstan-baseline.neon.
  6. Remove any spelling words specific to the extension from the dictionaries.
  7. Add the extension to the relevant removed list, either DRUPAL_CORE_REMOVED_MODULE_LIST or DRUPAL_CORE_REMOVED_THEME_LIST, in system.install.
  8. Check for references in @todo.

Release notes snippet

The Settings Tray module has been removed from core, and can now be installed as a contrib module,

Issue fork drupal-3586218

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.

quietone’s picture

quietone commented
Issue summary: View changes

quietone opened merge request !16578

catch’s picture

catch
he/him
Primary language English
commented
Status: Postponed » Needs work

With #3576667: [meta] Tasks to deprecate Settings Tray in main it should be possible to get to a green MR here. Still a couple of deprecation steps pending.

quietone opened merge request !16709

quietone’s picture

quietone commented
Status: Needs work » Needs review
smustgrave’s picture

smustgrave commented
Status: Needs review » Needs work

There's still mention in the .deprecations-ignore.txt file and maintainers entry. Rest looks good.

quietone’s picture

quietone commented
Status: Needs work » Needs review

@smustgrave. thanks. I added more instructions to the 'how to deprecate' an extensions docs.

smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Awesome looks good! Congrats for knocking another one out.

  • catch committed ee7c1cd8 on main 
    task: #3586218 Remove the Settings Tray module

By: quietone
By:...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to main, thanks! This is a small module but considering we realised when evaluating it for removal that no-one was sure what it does and that it doesn't work, it's a good one to remove, and the ideas it introduced were long moved out of the module itself.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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