πŸ‡¬πŸ‡§United Kingdom @kimberleycgm

Account created on 16 March 2015, about 10 years ago
  • Senior Backend Developer at ZoochaΒ 
#

Merge Requests

More

Recent comments

πŸ‡¬πŸ‡§United Kingdom kimberleycgm

kimberleycgm β†’ changed the visibility of the branch 3515571-d11-sessionmanager-service to hidden.

πŸ‡¬πŸ‡§United Kingdom kimberleycgm

Pushed an update. The payment type is now mandatory and we run through the parent create method for all the set up.

πŸ‡¬πŸ‡§United Kingdom kimberleycgm

Noticed one thing that needs adjusting - adding fix shortly.

πŸ‡¬πŸ‡§United Kingdom kimberleycgm

I've added an MR to update the composer restriction. I've tested with our site locally and it all works well, so I don't think there is any need to limit it to version 2.

I've had to use composer aliases to install commerce v3 in the meantime, so it would be great to get this released properly asap.

Thanks

πŸ‡¬πŸ‡§United Kingdom kimberleycgm

I've opened a MR adding the packages back to composer.json. I believe, without using the composer merge plugins, all the required packages need to be included in the main composer.json because submodule versions aren't read and used when those submodules are enabled.

The patch won't fix the problem while it's still a patch because patches are applied after the composer.json is read during composer install, so for now I've had to also require the packages directly in my project's composer.json.

πŸ‡¬πŸ‡§United Kingdom kimberleycgm

kimberleycgm β†’ made their first commit to this issue’s fork.

πŸ‡¬πŸ‡§United Kingdom kimberleycgm

I've also run into this problem and agree it should be configurable in some way, but open to suggestions on the best way. A `hook_entity_type_info_edit_form_modes_alter` perhaps, or it being defined as part of the form mode. I'm happy to get started on a solution.

πŸ‡¬πŸ‡§United Kingdom kimberleycgm

I've resolved merge conflicts with 2.x and have tested this along with the change from Issue 3050027 πŸ› Inline Blocks on Layout Builder Fail to Clone Correctly Needs work mentioned in #12. 3050027 alone didn't fix the issue for me so this is still needed. We're using paragraphs_asymmetric_translation_widgets in our setup, for what it's worth. We also encountered data loss with this so would be grateful to see a release with both fixes.

Attaching the patch file too ( https://www.drupal.org/docs/develop/git/using-gitlab-to-contribute-to-dr... β†’ ).

πŸ‡¬πŸ‡§United Kingdom kimberleycgm

Sorry - hit the issue form button by habit! Patch attached.

πŸ‡¬πŸ‡§United Kingdom kimberleycgm

kimberleycgm β†’ changed the visibility of the branch 3424715-repeated-languageinterfacelangcodenotspecified to hidden.

πŸ‡¬πŸ‡§United Kingdom kimberleycgm

I actually found that this fix didn't load the file for anonymous users. The patch in https://www.drupal.org/project/google_tag/issues/3153800 β†’ seems to fix both issues so far.

πŸ‡¬πŸ‡§United Kingdom kimberleycgm

This is working well for me. I've also attached the static file.

πŸ‡¬πŸ‡§United Kingdom kimberleycgm

I've updated the merge request to use the `??` syntax, and uploaded a diff file in case needed in that format

πŸ‡¬πŸ‡§United Kingdom kimberleycgm

kimberleycgm β†’ made their first commit to this issue’s fork.

Production build 0.71.5 2024