Verify all Drupal CMS recipes should be available in Project Browser post launch

Created on 11 December 2024, 5 months ago

Problem/Motivation

Currently the only projects allowed in Project Browser in Drupal CMS are:

            - drupal_cms_events
            - drupal_cms_blog
            - drupal_cms_case_study
            - drupal_cms_forms
            - drupal_cms_news
            - drupal_cms_person
            - drupal_cms_project
            - drupal_cms_seo_tools
            - drupal_cms_accessibility_tools

All of the recipes that are in the installer are there.

        "drupal/drupal_cms_blog": "Sets up a basic blog, with tagging.",
        "drupal/drupal_cms_case_study": "Adds a case study content type that provides detailed information about a subject to the intended audience.",
        "drupal/drupal_cms_events": "A content type and view to help you manage events.",
        "drupal/drupal_cms_news": "Adds a news content type with an overview page and related news items.",
        "drupal/drupal_cms_person": "Adds a person profile content type that provides information about a person.",
        "drupal/drupal_cms_project": "Adds a project content type that provides information about an example of work to the intended audience."
    }

The following recipes are applied by default. Should these be allowed to be re-applied? If so, they should be in project browser:

  - core/recipes/core_recommended_maintenance
  - core/recipes/core_recommended_performance
  - core/recipes/document_media_type
  - drupal_cms_admin_ui
  - drupal_cms_anti_spam
  - drupal_cms_authentication
  - drupal_cms_page
  - drupal_cms_privacy_basic
  - drupal_cms_remote_video
  - drupal_cms_seo_basic
  - easy_email_express

That leaves the following recipes:

drupal_cms_ai
drupal_cms_analytics
drupal_cms_content_type_base (dependency, probably doesn't make sense.)
drupal_cms_image
drupal_cms_multilingual
drupal_cms_search

drupal_cms_starter (will the world explode if you re-apply Drupal CMS to Drupal CMS?)

Proposed resolution

Decide if we want the following recipes to be re-apply-able:
- core/recipes/core_recommended_maintenance
- core/recipes/core_recommended_performance
- core/recipes/document_media_type
- drupal_cms_admin_ui
- drupal_cms_anti_spam
- drupal_cms_authentication
- drupal_cms_page
- drupal_cms_privacy_basic
- drupal_cms_remote_video
- drupal_cms_seo_basic
- easy_email_express
- drupal_cms_content_type_base
- drupal_cms_starter
- drupal_cms_image

Add the following to Project browser if they are not being excluded for certain reasons.
- drupal_cms_ai
- drupal_cms_analytics
- drupal_cms_multilingual
- drupal_cms_search

Remaining tasks

User interface changes

API changes

Data model changes

πŸ“Œ Task
Status

Active

Component

Base Recipe

Created by

πŸ‡ΊπŸ‡ΈUnited States thejimbirch Cape Cod, Massachusetts

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @thejimbirch
  • πŸ‡ΊπŸ‡ΈUnited States ultimike Florida, USA

    drupal_cms_search is a weird one because it appears in the drupal_cms_starter recipe's composer.json file but not in the recipe.yml file - maybe this is an oversight?

    I agree that drupal_cms_ai, drupal_cms_analytics, and drupal_cms_multilingual should all be available in Project Browser.

    -mike

  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts

    These things aren't oversights.

    • drupal_cms_ai and drupal_cms_analytics need input at apply time, and Project Browser has no UI for that yet. (There is a proof of concept in progress but it hasn't been committed or released in Project Browser.) AI in particular is tricky because its API keys need special handling that isn't supported by the recipe system.
    • drupal_cms_search might be applied by the starter recipe, which is why it's in the composer.json file. I think there are one or two blockers before we do that, though. If it doesn't make it into the starter recipe by release time, I'm guessing it will either be added to Project Browser, or hidden outright. This is a product-level decision.
    • drupal_cms_multilingual is not released at all, and it isn't part of the current Drupal CMS build. There is no project for it. This was due to reasons very much beyond our control (the track lead became indefinitely unavailable and hasn't, as far as I know, been replaced yet).
  • πŸ‡ΊπŸ‡ΈUnited States thejimbirch Cape Cod, Massachusetts

    Thanks, Mike and I were chatting and we didn't want to assume.

    1. So is https://git.drupalcode.org/project/drupal_cms/-/tree/1.x/recipes/drupal_... from the prototype? Should we remove that altogether?

    2. Probably a product-level decision, but do we want to allow for the recipes that have already been applied to be re-applied? This could be useful if you (or your AI agent) messed something up.

  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts

    So is https://git.drupalcode.org/project/drupal_cms/-/tree/1.x/recipes/drupal_... from the prototype? Should we remove that altogether?

    It's largely from the prototype. There may have been a few changes since then. I discussed removal with @tim.plunkett and we decided against it, since once a new track lead is found, work on that track should hopefully resume. There didn't seem to be much sense in removing it; it's not doing any harm by its presence.

    Probably a product-level decision, but do we want to allow for the recipes that have already been applied to be re-applied? This could be useful if you (or your AI agent) messed something up.

    Well...there's no way to prevent a recipe from being reapplied. Project Browser doesn't expose the functionality, but that's a bug which is going to be fixed by ✨ Allow recipes to be re-"installed" (re-applied) inside PB Active .

  • πŸ‡ΊπŸ‡ΈUnited States ultimike Florida, USA

    So is the plan that both drupal_cms_ai and drupal_cms_analytics will be apply-able in Drupal CMS 1.0 only via the command line? Or is that TBD?

    -mike

  • πŸ‡¦πŸ‡ΊAustralia pameeela

    TBD but this is all currently by design. Search works functionally but has no styles, so we don't want users applying it until we've fixed that. Ideally we will have the recipe inputs available for v1 but even if we don't, I think we'll expose the recipe, perhaps just with a prompt telling them what to do next.

    But, definitely not an oversight. What is in PB now is what we want users to see. We will never want to expose the starter/base recipes in PB as part of Drupal CMS, it would be extremely confusing. Recipes that require them will depend on them behind the scenes. Perhaps at some future date when we have a killer UI, we would show recipes that come pre-applied (e.g. basic page) for users to browse or reapply, but this is not a priority for v1.

    As for multilingual, this definitely is out of scope for v1 but I think @phenaproxima wanted the project created for infra reasons. Whether we remove the reference to it altogether is a tech decision I guess.

Production build 0.71.5 2024