The module does not follow the naming convention of Drupal AI modules.

The machine name should be ai_recipe_ckeditor and not drupal_cms_ai_ckeditor.

The former would indicate it is a recipe that works with Drupal AI in general, meaning that it can be applied to any Drupal website.

The latter would mean, that it requires Drupal CMS.

Since the recipe should be available for all Drupal websites and not require Drupal CMS to function, consider changing the naming.

If you agree, let's either create a new project or ask the infrastructure team to change the machine name (don't know which one is better).

Comments

breidert created an issue. See original summary.

kelly.m.jacobs’s picture

@breidert Sure, I can just create a new project, it may be simpler that way.

Would recipe_ai_ckeditor be more appropriate? The recipe is meant to help setup the AI CKEditor module, so it seems like the ai_ckeditor machine name should stay intact.

kelly.m.jacobs’s picture

Assigned: Unassigned » kelly.m.jacobs
kelly.m.jacobs’s picture

Assigned: kelly.m.jacobs » Unassigned
robloach’s picture

Status: Active » Needs work

ai_recipe_ckeditor

arianraeesi’s picture

robloach’s picture

Component: Code » Miscellaneous
Status: Needs work » Needs review

Rumour from Christoph has it that we're actually okay as is!

robloach’s picture

Assigned: Unassigned » robloach
Status: Needs review » Reviewed & tested by the community

Going to close this as Fixed for now.

robloach’s picture

Assigned: robloach » Unassigned
Status: Reviewed & tested by the community » Fixed

Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

arianraeesi’s picture

Status: Fixed » Closed (fixed)