Cape Cod, Massachusetts
Account created on 7 March 2013, about 12 years ago
#

Merge Requests

More

Recent comments

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

There have been more changes since my lasy post that need to be documented.

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

Updated the change record to add that you can’t change those two keys.

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

Recipes added an "Extra" section in core. https://www.drupal.org/node/3504056 β†’

The idea is that modules, like Project Browser could consume information from recipes like display a message, or redirect to a page.

I don't know if any additional work has been done on Project browser yet, but this was added specifically for this use case.

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

Thank you all for your contributions!

@kattia's first commit on Drupal.org

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

I added link to Recipe Presentations and added a link to Martin's user profile.

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

Thanks so much for the contribution, and great to see you!

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

Thanks so much for your contribution!

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

Looks great, but needs a merge request.

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

thejimbirch β†’ changed the visibility of the branch 3505369-document-the-new to hidden.

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

Needs a Merge request and should be added to the table of contents also.

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

Moving to Drupal core, and postponing on the Drupal Marketplace work.

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

Moving to Drupal core. Merge request will need to be moved also.

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

Moving to Drupal core.

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

Moving to Drupal core into the Recipes system component.

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

That would be a great place to start, but I hope we can review all the issues.

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

The documentation page linked above can be edited by anyone with a Drupal.org profile. Once you've added a recipe, or many recipes, post a comment here so we can give issue credit.

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

Hmmm, not working and I don't know why. Need to pause on this for now.

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

Looks good. Marking as RTBC. Will add a change record.

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

Discussed at DrupalCon to use the name setEntityProperties and use the simpleConfigUpdate style. set and setMultiple will remain, but the new action will be able to handle all the syntax.

config:
  action:
    node.type.article:
      setEntityProperties:
      # Can work on singular
      foo: bar
      # multiple
      cat: dog
      mouse: bird
      # and nested properties.
      nested.thing: yes
πŸ‡ΊπŸ‡ΈUnited States thejimbirch Cape Cod, Massachusetts

This looks good to me today and all comments resolved. Marking as RTBC.

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

Thanks for that, brain slip. GitLab.

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

I have no strong opinions either way, as both recipes are always applied, but revisions on content types is only one use. Another is in the config sync screen.

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

Closing this as Won't do.

But please share your recipe when you create it.

https://www.drupal.org/docs/extending-drupal/contributed-modules/contrib... β†’

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

Since we have revisions for content types turned on, IMHO, we should have the diff module to compare revisions.

Assigning to Pam for approval.

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

Moving back to Needs work as the PHP tests are failing.

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

thejimbirch β†’ created an issue.

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

I ran into the same issue, with images vanishing when I dragged them to a new position in the editor.

We weren't using linked images, but I had the same experience.

Unchecking "Force pasting as plain text" just fixed it for me.

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

Screenshot added to change record.

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

Change record added.

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

Change record added.

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

Added a screenshot and published the change record.

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

Change record updated and published.

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

Change record updated and published.

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

Updating page status.

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

One thing keep in mind is that set() does not offer the same functionality, specifically it can only replace a whole top-level property, while SimpleConfigUpdate supports the usual dot based syntax to set child keys directly. Maybe set() should be expanded to have similar capabilities.

As simpleConfigUpdate is in use by most recipe authors on config entities, I feel like we need a suitable replacement that as @berdir points out in comment #16

Updating the title of this issue with the current direction.

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

Changes looks good. Marking as RTBC assuming the tests pass.

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

Postponing based on comment in #10

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

All images were changed to webp except for:

/recipes/drupal_cms_case_study/content/file/drupal-cms-logo.png
/recipes/drupal_cms_project/content/file/drupal-cms-logo.png

The test failure was unrelated, all green now. Moving to RTBC, thanks for your contribution!

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

When a recipe installs a module, the simple config from that module is imported. So you can't import your own version of that config because it will already exist. Failures abound, config explosions everywhere!%&#@^&$^&!

Instead, in each of the recipe's config actions section, use the simpleConfigUpdate action to rewrite them.

config:
  actions:
    tagify.settings:
      simpleconfigUpdate:
        set_default_widget: true
πŸ‡ΊπŸ‡ΈUnited States thejimbirch Cape Cod, Massachusetts

The Starter Templates idea:
Driesnote: Portland DrupalCon 2022
Dries Buytaert
https://youtu.be/Ig676RzJbLo?t=3180

The first recipes presentation
MAKERS & BUILDERS - Drupal Recipes: From #Driesnote to Initiative
Alex Pott
https://www.youtube.com/watch?v=uNBRLS5zTa8

Drupal Distributions and Recipes Initiative Update
ltrainjohnson, robert-arias
DrupalCon Portland 2024
https://www.youtube.com/watch?v=320Nffq4f44

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

thejimbirch β†’ created an issue.

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

As the maintainer of the User Guide I can say that it's a bit of a one off approach, and I'm not sure we should replicate it again.

Thanks for the input. We are going to shift to the mkdocs and use Gitlab pages for publishing.

The https://www.drupal.org/project/ai β†’ has done this, so we can use their code as a guide.

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

thejimbirch β†’ changed the visibility of the branch 3409318-update-recipes-documentation to hidden.

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

thejimbirch β†’ changed the visibility of the branch 11.x to hidden.

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

Fixed converts to closed fixed after a two week period automatically. Just FYI

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

The images need to have their original file name remain the same as they are referenced from default content config entities. If the images file names (and/or extensions) are renamed, then the content entity that references them needs to be updated also.

This MR also has only one image updated. The issue summary mentions "images", but did not provide detail. Running the following command in the recipes folder reveals the following images:

find "$PWD" -type f | grep -Ei '\.(jpg|jpeg|png|gif|bmp|tiff)$'

/recipes/drupal_cms_events/content/file/DrupalCon-Atlanta.png
/recipes/drupal_cms_news/content/file/unsplash_John_Schnobrich.png
/recipes/drupal_cms_case_study/content/file/Government-building-EU-flag-credit-pexels-gintare-kairaityte-11923111-17178100.jpg
/recipes/drupal_cms_case_study/content/file/drupal-cms-logo.png
/recipes/drupal_cms_person/content/file/unsplash_Leilani_Angel.png
/recipes/drupal_cms_blog/content/file/DrupalCon-Barcelona-2024 - credit-Bram-Driesen.jpg
/recipes/drupal_cms_project/content/file/pexels-israel-franca-1108173-2101030.jpg
/recipes/drupal_cms_project/content/file/drupal-cms-logo.png
/recipes/drupal_cms_starter/content/file/drupal-cms-hero.png

Moving to Needs work.

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

This is a duplicate of πŸ› Trash link is a 404 on install because no entity types are enabled Active and is fixed in the next release.

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

It would be great if, once installed, entity reference fields used the Tagify widget by default instead of the standard autocomplete.

In ✨ SEO Tools recipe should automatically reapply itself every time a content type is created Active we are talking about a recipe that leaves behind an ECA that can applies config actions. So instead of having to re-apply a recipe, to go the SEO fields on new content types in that example, on new content type creation, the ECA applies config actions LIKE a recipe would. If this proves to be possible, I assume it could be used any time a reference field was created.

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

I'm not sure I agree that altering the source image should be the path here. Shouldn't the image tools creative derivative images create the most optimized images? I haven't had a Photoshop license since Photoshop 3. I also encourage our clients to upload large source image to future proof their content.

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

thejimbirch β†’ created an issue.

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

Thank you!

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

We could surely adjust it so that we don't expose the checkbox -- which would have no effect anyway -- but we do keep the disclosure, or some re-worded version of it.

I like this idea. It provides the message and the user can stop the installation if they don't agree. And we don't have to rely on another package or core to make any changes.

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

The code is available in standard drupal/core however there is a prompt to reject the tracking while Drupal CMS removed the prompt and disclosure.

Could you point out where this prompt is and how Drupal CMS is removing it? That may help with some options to solve this.

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

Is there a list of attendees to credit?

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

The launcher was removedand the documentation was updated in ✨ Replace the launcher script with better documentation Active . Closing as duplicate.

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

Closing for no response and not much information to begin with.

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

Drupal CMS installs the https://www.drupal.org/project/automatic_updates β†’ which enables core's updates module.

Should this issue be moved there?

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

Adding Platform.sh Certified developer badge.

Production build 0.71.5 2024