Problem/Motivation

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

The removal of Claro was approved in #3576460: [policy, no patch] Deprecate and remove Claro.

Remaining tasks

Postponed on #3621198: Change remaining usages of Claro

  1. Create a section on Deprecated and obsolete modules and themes to provide recommendations for sites using extension Claro 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-claro

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

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 opened merge request !15265

smustgrave made their first commit to this issue’s fork.

catch’s picture

catch
he/him
Primary language English
commented
Status: Postponed » Needs review
smustgrave’s picture

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

Rebased and all green. Since quietone did all the work here think I'm good to RTBC. Going to work on the subtree split once this lands.

  • godotislate committed 2aa15117 on main 
    task: #3576670 Deprecate Claro theme

By: quietone
By: smustgrave
godotislate’s picture

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

Committed 2aa1511 and pushed to main. Thanks!
Per @catch on Slack, this needs to wait for #3621198: Change remaining usages of Claro to be backported to 11.x before this can be backported.

Also, https://www.drupal.org/node/3223395 needs the Claro entry created and moved from approved. We may also want to update the note there for Seven to remove references to Claro.

quietone’s picture

quietone commented

All the documentation is done in the parent issue, there are items for that in the issue summary.

godotislate’s picture

godotislate
he/him
commented
Issue tags: -Needs documentation updates

OK, I was just going off the Remaining tasks in the IS here.

quietone opened merge request !17187

quietone’s picture

quietone commented
Issue summary: View changes

Oh, I see. Well the good news is that I plan to review all the templates for removing an extension from core and moving it to contrib. I hope to do that in the first quarter of next year.

smustgrave’s picture

smustgrave commented

Will need #3621198: Change remaining usages of Claro to land in 11.x first I think.

smustgrave’s picture

smustgrave commented
Status: Patch (to be ported) » Reviewed & tested by the community

that one landed and rebased this :)

  • catch committed c42dbd56 on 11.x 
    task: #3576670 Deprecate Claro theme

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

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

Committed/pushed to 11.x, thanks!

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.

smustgrave’s picture

smustgrave commented

Someone mind confirming the versions on the CR to publish?