Problem/Motivation

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

The contrib project is Tour

The removal of extension Tour was approved in #3325445: [Policy] Remove tour module from core

Remaining tasks

Postponed on all other issues which include a stable release in the contrib project.

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

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.

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

smustgrave opened merge request !6648

smustgrave’s picture

smustgrave commented
Issue summary: View changes
Status: Postponed » Needs review

Update

smustgrave’s picture

smustgrave commented

Not sure who can update https://www.drupal.org/node/3223395 but I have reserved a contrib space at https://www.drupal.org/project/tour

quietone’s picture

quietone commented
Status: Needs review » Postponed

@smustgrave, thanks for creating the MR as well as the contrib project. I recently added a link to the contrib project to the issue summary of the parent issue.

That is a wiki page and anyone can update it. Tour is already listed in the https://www.drupal.org/node/3223395#s-upcoming-deprecations section. Of course, that will have to change when this issue is committed. It will move to the section of already deprecated extenstions. But I am still considering how to indicated the core version when an extension was removed.

This is postponed on a stable release of the contrib project and #3405660: Remove tour from themes.

quietone’s picture

quietone commented

Oh, I have more reading to do. #3405660: Remove tour from themes is now postponed. I will get back to this later today.

smustgrave’s picture

smustgrave commented

so how come this one is postponed?

quietone’s picture

quietone commented
Issue summary: View changes

@smustgrave, I explained the reason in #6. Also, in the meta it is step 4 which is before step 5, which is this issue..

smustgrave’s picture

smustgrave commented
Status: Postponed » Needs review

Did a release https://www.drupal.org/project/tour

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

The only remaining thing is AFAICT to update the docs page that the change record points to, because that page should now point to the contrib module's project page.

The merge request looks great 😊

  • catch committed 1b4af7c4 on 10.3.x 
    Issue #3405672 by smustgrave, quietone: Deprecate Tour module

(cherry...

  • catch committed 21583269 on 11.x 
    Issue #3405672 by smustgrave, quietone: Deprecate Tour module
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

MR looks great indeed.

Committed/pushed to 11.x and cherry-picked to 10.3.x, thanks! I un-postponed #3421789: Ensure that Tour does not get special core treatment.

The documentation move is listed in #3376099: [11.x] [Meta] Tasks to remove Tour so we will probably update references there rather than here.

quietone closed merge request !6648

Status: Fixed » Closed (fixed)

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