Account created on 21 April 2022, almost 3 years ago
  • Backend Developer at iO 
#

Merge Requests

More

Recent comments

🇧🇪Belgium cedricl

Tested it and seems to not be giving issue anymore

🇧🇪Belgium cedricl

Needs review. And should also be changed in the description of the module by a maintainer

🇧🇪Belgium cedricl

Wow, no idea how I didn't see that. That's exactly what we needed. Thanks

🇧🇪Belgium cedricl

Looks good, tested it and no errors occured anymore

🇧🇪Belgium cedricl

Changes in the fork, should be reviewed

🇧🇪Belgium cedricl

This is working for me. Thanks for the update

🇧🇪Belgium cedricl

We also did something like this but with support for custom attributes. Instead of just only adding the default attributes for the module. The custom attributes are 'style' and 'type'. But this can be everyting since we loop over the definitions from the \Drupal\link_attributes\LinkAttributesManager

🇧🇪Belgium cedricl

Turned out te be some obscure (internal) module we had enabled that was changing how the breadcrumbs work. Disabled it and all worked. Closing the issue.

🇧🇪Belgium cedricl

It seems like it only returns breadcrumbs if the node is in my a menu, but i always want a breadcrumb not only when it's in a menu. I think it is core related, will look into it. And give an update if I find it.

🇧🇪Belgium cedricl

Also noticed this issue after updating form 2.1.3 to 2.1.4. I downgraded for now, but might be a good thing to look at changes made from 2.1.3 to 2.1.4.

🇧🇪Belgium cedricl

Updated patch to not make entity type behaviors required in graphQL

🇧🇪Belgium cedricl

Added an empty route. This fixed the issue. Changes are in the fork.

🇧🇪Belgium cedricl

hi almunnings,

I'm not really a fan of the tokens method since, i like my data spread in seperate query instead of everything in the pageInfo, but this indeed covers most use cases for users. I'll stick to using the edges method i found for now. If i find the time I will update the module so we don't have to use a UUID for a single query. Can be put on hold for now

🇧🇪Belgium cedricl

I tested patch https://www.drupal.org/files/issues/2023-10-16/2422737-37.patch from https://www.drupal.org/project/drupal/issues/2422737 📌 Tabledrag event after dragging elements Needs work and for me this fixes the issue.

🇧🇪Belgium cedricl

Ticket should be reopened. The drag and drop feature for paragraphs still seems to not work. My bad for only reading the title. The auto reload works on changing selects and textareas, so this part is done. But when draging and dropping paragraph the preview doesn't auto reload. And Even when I press the 'preview' button manually it doesn't update. But after updating a random input field it reloads, and also shows the order changes of the paragraphs.

🇧🇪Belgium cedricl

The patch file fixed the issue. Thanks for the fast responses.

🇧🇪Belgium cedricl

Updated patch to keep in line with update graphql compose module.

🇧🇪Belgium cedricl

Added it as a seperate module, since not all sites have multiple languages and therefor don't need this field. The availableLanguages field has a language and url field, but can be extended if needed.

🇧🇪Belgium cedricl

Tested the MR, seems to work. Could this be merged?

🇧🇪Belgium cedricl

Tested your last changes, can be merged

🇧🇪Belgium cedricl

When you have a node and paragraph with the same name, it only adds the entite type behaviors field to the content type in the graphql schema.

With patch v3, it gets added to both the paragraph and the content type with the same name.

🇧🇪Belgium cedricl

I provided a patch so you don't have to put things in the settings file like this $config['graphql_compose.settings']['field_config']['paragraph']['YOUR_PARAGRAPH_TYPE']['FIELD_YOUR_FIELD_NAME']['required'] = FALSE;
But now you can do it the schema configurer, i've added a checkbox required which by default is set the same as the settings of your field. But you can override it and make that field not required for the schema in GraphQL.

When saving it, the required value gets added to the config.

Not sure if this breaks anything else, but for our use case i haven't seen it breaking something. Should be tested and reviewed

🇧🇪Belgium cedricl

We did this, but made it hardcoded only for the fields we needed, you can take a look in our module for inspiration.

🇧🇪Belgium cedricl

It looks like the changes that happened in your merge request are removed from 2.1, in a later commit. I added a patch to re-include these changes, so the preview with paragraphs works again.

🇧🇪Belgium cedricl

Code works as intented. Can be merged.

Thanks Almunnings!

🇧🇪Belgium cedricl

This seems to be related.
https://www.drupal.org/project/decoupled_preview/issues/3398030 🐛 Issue With Paragraph Entity Active

🇧🇪Belgium cedricl

The translations are coming through for items, but the field 'name' isn't being translated. I added the translation for the title of the menu here.
/en/admin/structure/menu/manage/MENU_ID/translate/fr/edit. But when querying the menu in french it still gives the label in the original menu language (Dutch) but the items are in french as should be.

Extra visual info: https://imgur.com/a/RCY8rGp

Ticket should be re-opened.

🇧🇪Belgium cedricl

Turned out my metatag defaults config files got removed from my site. No clue why this happened, but after adding this the issue was fixed. Ticket can be closed

🇧🇪Belgium cedricl

Does exactly what I wanted. Thanks for the help @almunnings

🇧🇪Belgium cedricl

Added schema extension for behaviors in nodes.

🇧🇪Belgium cedricl

CedricL made their first commit to this issue’s fork.

🇧🇪Belgium cedricl

#3 does the trick. Thanks for the help!

🇧🇪Belgium cedricl

For drupal core 9.5.x i'm getting
Cannot apply patch https://www.drupal.org/project/drupal/issues/3101671#comment-14976504 (https://www.drupal.org/files/issues/2023-03-21/3101671-21-against-deletable-9.5.x.patch)!
Any fixes for this issue. I also applied the patches from #22

🇧🇪Belgium cedricl

Fixed issues for the keycloak groups. Some keys in the config file changed names, so did the config file itself. There are still others that need to be changed. But keycloak group mapping is working with this patch.

Production build 0.71.5 2024