Problem/Motivation

In #3439923: Add recipes api as experimental API to core we added 4 recipes APIs to core. As a follow up, we need to identify and add subsystems on Drupal.org and subsystem maintainers in MAINTAINERS.txt. These maintainers have special roles in the governance structure and additional permissions, like the ability to adjust the Assigned field on behalf of others, assign issue credits in core, etc. See https://www.drupal.org/contribute/core/maintainers#subsystem for what this means.

The new APIs added were:

  • Drupal\Core\Config\Action
  • Drupal\Core\Config\Checkpoint
  • Drupal\Core\Recipe
  • Drupal\Core\DefaultContent

In this conversation in the Recipes channel of Drupal Slack, @alexpott determined that we need two subsystems, ‘recipe system’ and ‘default content system’. The Config Actions and Config Checkpoint issues can live in existing systems.

I think config actions can stay in config or config entity system… but I think we need ‘recipe system’ and ‘default content system’

Proposed resolution

  • Add ‘recipe system’ and ‘default content system’ subsystems to Drupal.org.
  • Identify the subsystem maintainers.
  • Provide a patch to MAINTAINERS.txt (committers will do the followup tasks of assigning maintainer permissions, etc).

Remaining tasks

Do it.

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3447072

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

thejimbirch created an issue. See original summary.

thejimbirch’s picture

Issue summary: View changes
thejimbirch’s picture

In the same slack thread, @alexpott identified the maintainers as

@alexpott
@phenaproxima
@thejimbirch

I also think we need a core issue to add these two sub systems to core and make us 3 maintainers.

thejimbirch’s picture

alexpott’s picture

@thejimbirch thanks for working on this. I'm happy to be in MAINTAINERS.txt for default content and recipes.

alexpott’s picture

Status: Active » Needs review
dww’s picture

Looks good to me, though we probably want @phenaproxima to formally accept in here before this is committed.

phenaproxima’s picture

I formally accept co-maintainership of the recipe subsystem.

dww’s picture

Status: Needs review » Reviewed & tested by the community

Groovy, thanks. Ship it! 🎉

longwave’s picture

Status: Reviewed & tested by the community » Fixed

Thanks for volunteering to maintain these new subsystems! I've added the two new subsystems to the component dropdown.

Committed and pushed a885cc86c5 to 11.x and 7df7eb4c51 to 11.0.x and 8a26f33828 to 10.4.x and 47794d4077 to 10.3.x. Thanks!

  • longwave committed 47794d40 on 10.3.x
    Issue #3447072 by thejimbirch, alexpott, phenaproxima: Add 'recipe...

  • longwave committed 8a26f338 on 10.4.x
    Issue #3447072 by thejimbirch, alexpott, phenaproxima: Add 'recipe...

  • longwave committed 7df7eb4c on 11.0.x
    Issue #3447072 by thejimbirch, alexpott, phenaproxima: Add 'recipe...

  • longwave committed a885cc86 on 11.x
    Issue #3447072 by thejimbirch, alexpott, phenaproxima: Add 'recipe...
longwave’s picture

longwave’s picture

Version: 11.0.x-dev » 10.3.x-dev

Status: Fixed » Closed (fixed)

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