🇬🇧United Kingdom @3li

No longer working on 10.5.x
Get the following errors in the console:

• Uncaught TypeError: Cannot read properties of undefined (reading 'config') at config.js?v=10.5.2:11:37
• Uncaught TypeError: Cannot read properties of undefined (reading 'config') at ckeditor5.js?t1nbxf:8:36
• Uncaught TypeError: Cannot destructure property 'check' of 'e.icons' as it is undefined. at math.js?t1nbxf:1:12386
• CKEditorError: plugincollection-plugin-not-found {"plugin":null} at ckeditor5-dll.js?v=45.2.0:5:160418

Looks to me like the config that is being used in javascript is no longer being defined (not sure why) and that math.js is trying to use something that no longer exists.
Then CKEditor5 errors because it cannot setup the plugin correctly and that causes CKEditor to stop loading.

Thank you for the suggestion, updated the text (moved things around a bit too)

3li → created an issue. See original summary → .

I think for this to be added to the module in my opinion it needs two things:

1. The placeholder text should be translatable
2. When the content becomes empty, the placeholder should show up again

I am not using Fast Autocomplete → but I had the same issue after upgrading to 3.0.x on the search page - even though that was not a view.

The patch has resolved this problem.

3li → changed the visibility of the branch 2846878-add-class-to-body to active.

3li → changed the visibility of the branch 2846878-add-class-to-body to hidden.

Works well for myself, just creating a patch for safe reference.

Re-rolled for latest dev

Changes made to change loc_id from unique to primary key.

fixing issue with #6

Resolves the issue for me.
Drupal 10.3.x with workspaces.

Same patch but for version 4 of graphql twig.

Following on from #243 about Session not set warning.
I have added a patch that seems to remove the warning for myself.

I do have a feeling this should be added to another issue though.

I found that #16 does work, but due to the limitations of patches you cannot apply changes to composer.json to say install/update another modules requirements - https://docs.cweagans.net/composer-patches/troubleshooting/non-patchable...

I have made some changes to #16 to allow the fields to be empty and should not cause any issues.

Steps to apply:

1. Install hubspot module 3.x: composer require 'drupal/hubspot:3.x-dev@dev'
2. Alter composer.lock hubspot-php requirement:
   From:
   

   "require": {
                   "drupal/core": "^8.9 || ^9 || ^10",
                   "drupal/webform": "*",
                   "hubspot/hubspot-php": "^3.0"
               },

   To:

   "require": {
                   "drupal/core": "^8.9 || ^9 || ^10",
                   "drupal/webform": "*",
                   "hubspot/hubspot-php": "^5.2"
               },

3. Install hubspot-php 5.2: composer require hubspot/hubspot-php:^5.2 -W
4. Apply patch #16/#22

This has fixed the Drupal 10.3 issues but the tests are not working still.

The CI pipeline is now working for the previous version 10.2.x, see 🐛 Problem when using graphql_twig with twig 3.9.0+ Active about fixes for 10.3.x

3li → changed the visibility of the branch 3463686-fix-ci-pipeline to active.

3li → changed the visibility of the branch 3463686-fix-ci-pipeline to hidden.

3li → created an issue.

3li → changed the visibility of the branch 3463682-twig-3.9 to active.

3li → changed the visibility of the branch 3463682-twig-3.9 to hidden.

3li → created an issue.

re-roll for latest version

Fixing a small bug.

Fixing typo in the field name.

3li → created an issue.

This patch is built on top of ✨ Language preferred and Canonical settings missing Needs work so assuming that is used in dev it should apply nicely.

3li → created an issue.

Added in some tests, added default language, updated description for language for some examples and added a https warning for canonical urls.

Think I found the problem.
We should be running an extra process facets_exposed_filter_remove_validation (facets/modules/facets_exposed_filters/src/Plugin/views/FacetsViewsPluginTrait.php) which is meant to unset($element["#needs_validation"]);

However for some reason when using select2 nothing gets created and thus this process does not get attached or run, meaning that validation is still enabled.

3li → created an issue.

This fixes the issue for myself, I have just created a patch version of it.

3li → created an issue.

Think this and other PHP 8.2 issues have been resolved here: 📌 PHP 8.2 compatibility Fixed

Patch applied to dev.

Added the patch to dev, also added another fix on top that removes the char limit for email messages (as it only really needed for slack).

Thanks for the fix.

3li → created an issue.

Resolved in 2.x

Trying to fix test errors and resolving an issue that prevents the correct handler being used on processing.

3li → created an issue.

Re-roll to make compatible with Drupal 10.

3li → created an issue. See original summary → .

Will be resolved in 📌 Drupal 10 Ready Needs review

3li → created an issue.

3li → made their first commit to this issue’s fork.

Tests should be written to cover the new supported entities.

Fixed during 📌 Setup GitLab CI Active

As this ticket has been extended by ✨ Add support for block_content and taxonomy_term name fields (core) Needs review I am going to close this one as duplicate.

3li → made their first commit to this issue’s fork.

Resolved apart of ✨ Make Unique Field Ajax 2.x compatible to Drupal 10 Fixed

Think this should be resolved now that media support has been enabled ✨ Media library support Fixed

Duplicate: #3312251: Unique Field Ajax php 8.1 - Deprecated function mb_strtolower →

Fixed apart of 📌 Setup GitLab CI Active

Fixed apart of 📌 Setup GitLab CI Active

3li → changed the visibility of the branch 3414797-setup-gitlab-ci to active.

3li → changed the visibility of the branch 3414797-setup-gitlab-ci to hidden.

3li → changed the visibility of the branch 3414797-setup-gitlab-ci to hidden.

3li → changed the visibility of the branch 3414797-setup-gitlab-ci to hidden.

Duplicate of 🐛 Deprecated hook_field_widget_form_alter() Needs review

3li → created an issue.

Think this is a duplicate of ✨ Media library support Fixed