Problem/Motivation

#3159848: [Policy] Always install Drupal with Standard on the UI, pared down of use case specific elements (content types, node listing, commenting, theme) left the tags vocabulary in standard profile / recipe, although it is free flowing and is not attached to anything that uses it. We kept it there to manage scope. We should remove it.

Steps to reproduce

Proposed resolution

Remove the tags vocabulary.

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

The standard install profile and recipe does not include the tags vocabulary anymore.

CommentFileSizeAuthor
#15 3588274-nr-bot_eubfac9y.txt91 bytesneeds-review-queue-bot

Issue fork drupal-3588274

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

gábor hojtsy created an issue. See original summary.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Parent issue: #3587118: Remove content types from the standard install profile and recipe » #3159848: [Policy] Always install Drupal with Standard on the UI, pared down of use case specific elements (content types, node listing, commenting, theme)
gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Active » Postponed

Postponed on #3159848: [Policy] Merge the minimal and standard install profiles into a starting point recipe landing first.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Postponed » Active
Issue tags: +Lean Starter Experience

The content types are removed on main in #3587118: Remove content types from the standard install profile and recipe, we can move on with this actially.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Active » Needs work

gábor hojtsy opened merge request !15809

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Needs work » Needs review

Green and looks complete :)

smustgrave’s picture

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

Wish I had this applied with the comments one, but no worries. Same deal standard install no tags field.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Parent issue: #3159848: [Policy] Always install Drupal with Standard on the UI, pared down of use case specific elements (content types, node listing, commenting, theme) » #3591027: [META] Remove use case specific elements from Standard and Node module. Always install with Standard. Keep Minimal for CLI use only.

Adjust parent to implementation issue from policy issue.

  • catch committed 1dab1d77 on main 
    task: #3588274 Remove tags vocabulary from standard profile and recipe...
catch’s picture

catch
he/him
Primary language English
commented
Version: main » 11.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed/pushed to main, thanks! Will need a backport for 11.x

catch closed merge request !15809

smustgrave opened merge request !15873

smustgrave’s picture

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

I asked Gabor if it was okay I did the backport and he said sure.

This was an easy backport only issue was a single migrate test in contact.

needs-review-queue-bot’s picture

needs-review-queue-bot commented
Status: Reviewed & tested by the community » Needs work
StatusFileSize
new 3588274-nr-bot_eubfac9y.txt91 bytes

The Needs Review Queue Bot tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Needs work » Reviewed & tested by the community
Issue tags: +11.4.0 release highlights

Was a clean rebase with no changes required, so back to RTBC.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Needs work

Needs a rebase again, @smustgrave can you maybe help with this?

smustgrave’s picture

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

Rebased!

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Title: Remove tags vocabulary from standard profile and recipe » Remove Tags vocabulary from standard profile and recipe
damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Does this need a change notice too?

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Title: Remove Tags vocabulary from standard profile and recipe » Remove Tags vocabulary from Standard profile and recipe
gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Issue tags: -11.4.0 release highlights +11.5.0 release highlights

@damienmckenna: good point, half of the related issues already had change notices. I just added a change notice to #3588276: Remove comment module dependency from standard profile and recipe and added one for this issue too at https://www.drupal.org/node/3606438 -- so far only targeted at main/12.0.0 but when it lands as backport it will also need to be added. I assume 11.5 at this point.