Problem/Motivation

Track the steps needed to deprecate extension Olivero. See Remove a core extension and move it to a contributed project of the deprecation policy.

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

All the child issues here, except in the infrastructure project, are "11.5.0 release priority".

Remaining tasks

To find uses of olivero:

    git grep -lwi olivero | grep -v core/themes/olivero/ | grep -v core/assets | grep -v phpstan-baseline | grep -v MAINTAINERS
  1. ✅Find someone to maintain the contrib version of the extension. mherchel has agreed to do this.
  2. ✅Add the extension to the list of the Upcoming Deprecations.
  3. ✅Move integrations implemented by other modules to the extension.
    Create child issues or child meta issues, as needed, to address the following points. Not all points will apply to all extensions.
  4. ✅Do a thorough search of core for any remaining references to the extension. If references are found, outside of the extension, then create issues to remove the references.
  5. ✅Create the contrib project with a stable release, before the alpha version of the major release. Follow the process in Create the contrib project with a stable release for creating the sub tree split.
  6. ✅Deprecate the core extension.#3595083: Deprecate the Olivero theme
  7. ✅Create an issue, 'Ensure that Olivero does not get special core treatment' in the packages.drupal.org issue queue to ensure that the composer namespace for the contributed project matches the core one after removal. This is to allocate the drupal/olivero Composer namespace to contrib extension rather than the core extension.
  8. ✅Update the Deprecated and obsolete extensions

Comments

quietone created an issue. See original summary.

quietone’s picture

Issue summary: View changes
quietone’s picture

Issue summary: View changes
quietone’s picture

quietone’s picture

Issue summary: View changes
quietone’s picture

Issue summary: View changes
quietone’s picture

Issue summary: View changes
quietone’s picture

There are two Nightwatch tests in the Search module which shouldn't have to be moved, as long as Search is removed before Olivero.

Still to do are 47 tests with that are likely installing Olivero

$ git grep -lwi "'olivero'" | grep -v core/themes/olivero/ | grep -v core/assets | grep -v phpstan-baseline | grep -v MAINTAINERS | grep -iv nightwatch | grep -i test | wc -l
47
quietone’s picture

After after apply the diffs in the child issues the word 'olivero' is found in 38 tests.

$ git grep -lwi olivero | grep -v core/themes/olivero/ | grep -v core/assets | grep -v phpstan-baseline | grep -v MAINTAINERS | grep -v core/.cspell.json | grep -v core/misc/cspell/drupal-dictionary | grep -v core/modules/shortcut | grep -v core/modules/search | grep -i test | wc -l
38

I made a child issue to investigate those, #3617584: Change tests using Olivero

xmacinfo’s picture

What will replace Olivero for regular Drupal installation? Drupal CMS will not be used on all installation.

gábor hojtsy’s picture

Nothing will be added in place of Olivero. We expect people installing Drupal core know what they are doing and if they need a frontend theme at all (ie. not a headless backend), they will pick a modern component based layout tool and component system to build their Drupal site with. Olivero is in the way of that. #3590882: Standard profile should not include Olivero anymore, should provide guidance on how to get started introduces a welcome page to explain the new situation to people who know Drupal or are new to Drupal core.

This is in line with the Drupal core strategy of prioritising being a tool for CMS builders with strong building blocks. See https://www.drupal.org/about/core/policies/drupal-core-strategy

quietone’s picture

Issue summary: View changes
quietone’s picture

Issue summary: View changes

Olivero has two search blocks in optional config. And when Search is removed there are two tests that fail \Drupal\KernelTests\Config\DefaultConfigTest and \Drupal\FunctionalTests\Theme\OliveroTest.

quietone’s picture

Issue summary: View changes

I didn't find any instances of Olivero in help files.

quietone’s picture

Issue summary: View changes

I asked the other rm's about how to handle the tests that fail due to the optional config for Search mentioned in #13. catch and xjm agree that the test(s) can have an #[IgnoreDeprecations] until the Olivero is removed.

quietone’s picture

Issue summary: View changes
quietone’s picture

Just updating the colors.

quietone’s picture

Issue summary: View changes
quietone’s picture

Issue summary: View changes
quietone’s picture

Issue summary: View changes
catch’s picture

Just refreshing issue statuses in the issue summary.

quietone’s picture

Issue summary: View changes
quietone’s picture

Issue summary: View changes
quietone’s picture

Issue summary: View changes

We try to remove extensions from the database dumps in a single issue, but Olivero was ready at the time, so doing later than usual.

quietone’s picture

Issue summary: View changes

mherchel has agreed to maintain the contrib version.

@mherchel, thanks

quietone’s picture

Issue summary: View changes
Status: Active » Fixed

There is a stable release of Olivero!

Thanks everyone for getting this done.

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.