Account created on 7 November 2007, over 16 years ago
#

Merge Requests

Recent comments

πŸ‡¬πŸ‡§United Kingdom Barry_Fisher

For what it's worth, I came across this issue and found the patch in #92 πŸ“Œ Migration Lookup plugin does not return multiple values when matched Needs work worked perfectly for me.

Thanks @shashikant_chauhan!

For added context (see below), I'm processing a JSON file with the url source plugin from the migrate_plus module. As the paragraph mappings come from a `paragraphs` subkey in the JSON structure, I had to do the following in a sub_process plugin. The issue was that only migration lookups only worked for the first migration listed: `blog_paragraphs_copy`. I know this because I tried swapping the order and the `blog_paragraphs_copy_media` then worked but `blog_paragraphs_copy` didn't.

With the patch in #92 πŸ“Œ Migration Lookup plugin does not return multiple values when matched Needs work both migration lookups are now working successfully. If an item in one migration is found, then it's used, otherwise the other is used.

field_structured_content:
    plugin: sub_process
    source: paragraphs
    process:
      target_id:
        -
          plugin: migration_lookup
          migration:
            - blog_paragraphs_copy
            - blog_paragraphs_copy_media
          source: id
        -
          plugin: extract
          index:
            - 0
      target_revision_id:
        -
          plugin: migration_lookup
          migration:
            - blog_paragraphs_copy
            - blog_paragraphs_copy_media
          source: id
        -
          plugin: extract
          index:
            - 1
πŸ‡¬πŸ‡§United Kingdom Barry_Fisher

@cilefen and @longwave - Thank you both for your feedback. You're right. It had us scratching our heads here. It would appear that we're using the (old) patch at: https://www.drupal.org/project/drupal/issues/3370828#comment-15130100 πŸ› Ensure that edge caches are busted on deployments for css/js aggregates Fixed

This is altering the AssetGroupSetHashTrait to resolve to "Ensure that edge caches are busted on deployments for css/js aggregates" issue.

As 3370828 πŸ› Ensure that edge caches are busted on deployments for css/js aggregates Fixed has now been resolved, I'll mark this as closed.

πŸ‡¬πŸ‡§United Kingdom Barry_Fisher

Hi Mark,

Patch is good and applies cleanly. It's a simple and logical fix, so hopefully this will get accepted by the maintainer.

Thanks!

πŸ‡¬πŸ‡§United Kingdom Barry_Fisher

Updating issue description based on latest updates I've made. After inspecting the module code I wanted to give more comprehensive to support to Linkit, views, and FieldFormatter.

Looking forward to feedback.

πŸ‡¬πŸ‡§United Kingdom Barry_Fisher

Thanks @mandclu for the quick review and thanks for your efforts with this module. It has been really useful on a local theatre site we've recently launched.

πŸ‡¬πŸ‡§United Kingdom Barry_Fisher

Merge request done. Awaiting feedback from the wider community and the maintainers.

In the meantime, here's a link to the patch file for composer patches:

https://git.drupalcode.org/project/smart_date/-/merge_requests/48.patch

Production build 0.69.0 2024