Problem/Motivation

Deprecate extension Olivero. See Remove a core extension and move it to a contributed project of the deprecation policy.

The removal of Olivero was approved in #3590816: [policy, no patch] Deprecate Olivero and move to contrib.

This is postponed until all usages of this extension outside of the extension itself is properly handled.

Remaining tasks

This is the actual deprecation and is limited to the following tasks. See the parent issue for the full process.

  1. Create a section on Deprecated and obsolete modules and themes to provide recommendations for sites using extension Olivero. The recommendations are to include instructions for sites using the extension and for contributed projects that depend on then extension.
  2. Add the following to the extension info.yml file
  3. lifecycle: deprecated
    lifecycle_link: https://www.drupal.org/node/3223395#s-olivero

  4. Add #[IgnoreDeprecations] to all tests in the extension.
  5. Add a change record and include a link the the doc page.
  6. Deprecate any library that is only used by the extension.
  7. 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-3595083

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:

Comments

quietone created an issue. See original summary.

quietone’s picture

catch’s picture

Status: Postponed » Needs review
daffie’s picture

Status: Needs review » Needs work

The CI pipeline is failing. I think there is still some Olivera tests.

gábor hojtsy’s picture

Status: Needs work » Needs review

#3622600: Remove remaining uses of Olivero in tests had what looked like the last Olivero tests, that JUST landed.

daffie’s picture

The CI pipeline is still failing.

gábor hojtsy’s picture

That's failing on #3623728: Node.js v24 upgrade fails all Nightwatch test runs unrelated to this issue, working on that now :)

catch’s picture

Status: Needs review » Reviewed & tested by the community

MR looks good. I only did a gitlab UI rebase. Moving to RTBC.

gábor hojtsy’s picture

Agree it looks good. Technically the attribute on OliveroHexToHslTest would not be needed but it will be removed sort of immediately anyway, so its fine.

  • larowlan committed 44cfb165 on main
    task: #3595083 Deprecate the Olivero theme
    
    By: quietone
    By: catch
    By:...

  • larowlan committed f57a99d2 on 11.x
    task: #3595083 Deprecate the Olivero theme
    
    By: quietone
    By: catch
    By:...
larowlan’s picture

Version: main » 11.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)
Issue tags: +Needs documentation updates

Committed and pushed 44cfb16574b to main. Thanks!

Backported to 11.x

Added a change record

We need documentation updates here (The link on d.o). Setting to patch (to be ported) for that. Please move to fixed when that is done.

quietone’s picture

Status: Patch (to be ported) » Fixed
Issue tags: -Needs documentation updates

The documentation updates are part of the parent issue, so removing tag.

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.