Problem/Motivation

People want the RecipeDiscovery class to do actual recipe discovery, but that's not the architecture - RecipeDiscovery is only for loading recipes included in other recipes.

See #3446354: Create a simple class to allow discovering recipes in the file system

Proposed resolution

Move the current RecipeDiscovery code into RecipeConfigurator class.

Remaining tasks

User interface changes

None

API changes

None - all classes a final and @internal.

Data model changes

None

Release notes snippet

N/a

Issue fork drupal-3447210

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

alexpott created an issue. See original summary.

alexpott’s picture

Status: Active » Needs review
jnicola’s picture

Status: Needs review » Reviewed & tested by the community

Tested locally with the recipe UI stuff I've got built, works!

  • longwave committed 2c3c9572 on 10.3.x
    Issue #3447210 by alexpott, jnicola: Move RecipeDiscovery into...

  • longwave committed 43ce451d on 10.4.x
    Issue #3447210 by alexpott, jnicola: Move RecipeDiscovery into...

  • longwave committed 31ff330a on 11.0.x
    Issue #3447210 by alexpott, jnicola: Move RecipeDiscovery into...
longwave’s picture

Version: 11.x-dev » 10.3.x-dev
Status: Reviewed & tested by the community » Fixed

I know this is all @internal but let's land this now before we cut a release that contains it.

Committed and pushed 2e3e874662 to 11.x and 31ff330aaf to 11.0.x and 43ce451db2 to 10.4.x and 2c3c957208 to 10.3.x. Thanks!

  • longwave committed 2e3e8746 on 11.x
    Issue #3447210 by alexpott, jnicola: Move RecipeDiscovery into...
longwave’s picture

Component: other » recipe system

And we can use the brand new core component for these issues now :D

Status: Fixed » Closed (fixed)

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