Account created on 17 July 2007, almost 17 years ago
#

Merge Requests

More

Recent comments

πŸ‡¨πŸ‡ΏCzech Republic parisek

I have similar issue and this MR fixes it for me (I'm using it exclusively for webform with AJAX enabled), thank you

greatmatter: Could you provide more information where it's not working (exact example) or steps to reproduce?

πŸ‡¨πŸ‡ΏCzech Republic parisek

I can confirm that it fixes my issue, so marking as duplicate, thank you

πŸ‡¨πŸ‡ΏCzech Republic parisek

It's webform, you can test it here https://almeva.swiss/pl/kontakty

Thank you

πŸ‡¨πŸ‡ΏCzech Republic parisek

This is still issue, patched worked for me

πŸ‡¨πŸ‡ΏCzech Republic parisek

Updated #168 πŸ› Untranslated menu items are displayed in menus Needs work patch to latest 10.2 - using "menu.language_tree_manipulator:filterLanguage" as I don't know how to migrate to current MR via $context which is locked to only allow SystemMenuBlock use.

πŸ‡¨πŸ‡ΏCzech Republic parisek

There is module https://www.drupal.org/project/jquery_once β†’  to fix this for legacy stuff

πŸ‡¨πŸ‡ΏCzech Republic parisek

I have this problem with older websites (missing views.view.media.yml), new installs are fine. Updated patch to latest Drupal.

πŸ‡¨πŸ‡ΏCzech Republic parisek

Updated the default installation variable. However, we might need to create an update hook for existing module users to streamline update to Drupal 10.1+

πŸ‡¨πŸ‡ΏCzech Republic parisek

Working for me, thank you

πŸ‡¨πŸ‡ΏCzech Republic parisek

We have same issue and #3 fixed it for us, thank you

πŸ‡¨πŸ‡ΏCzech Republic parisek

I found pretty good example how it could be implemented in this module (use EmailAdjuster) https://www.drupal.org/project/maillog/issues/3259616 ✨ Support Symfony Mailer RTBC

I think it's unnecessary to create separate module for this feature.

πŸ‡¨πŸ‡ΏCzech Republic parisek

xeniksp: Thank you for your patch, it fixed this issue, I created MR and hope that maintainer merge it soon.

πŸ‡¨πŸ‡ΏCzech Republic parisek

I get same error with current Drupal 10.1.5. Adjusted priority to critical because module is unusable.

πŸ‡¨πŸ‡ΏCzech Republic parisek

Seems really hard to rebase MR for current branch so closed it.

Found one issue - enclosure field is not translated we just get default language, probably we should implement EntityTranslationRenderTrait (like web/core/modules/views/src/Plugin/views/row/EntityRow.php)

πŸ‡¨πŸ‡ΏCzech Republic parisek

That sounds great if this module support TMGMT β†’ translations together. I changed Title and hope to more people will need this.

Thinking about
- checkbox on node edit form to choose if I need asymmetric translations or not
- use separate tab for asymmetric translations and on regular translation page show only notification

πŸ‡¨πŸ‡ΏCzech Republic parisek

rafmagsou, thank you for patch, I opened MR should be easier for mainainer to fix it.

πŸ‡¨πŸ‡ΏCzech Republic parisek

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

πŸ‡¨πŸ‡ΏCzech Republic parisek

Created MR and hid patches.

πŸ‡¨πŸ‡ΏCzech Republic parisek

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

πŸ‡¨πŸ‡ΏCzech Republic parisek

wangshy thank you, I got same error, created MR with fix inspired by your patch

ypeError: Drupal\Core\Entity\EntityDisplayBase::setComponent(): Argument #2 ($options) must be of type array, null given, called in /var/www/html/web/core/lib/Drupal/Core/Entity/EntityViewBuilder.php on line 530 in function Drupal\Core\Entity\EntityDisplayBase->setComponent() (line 340 file /var/www/html/web/core/lib/Drupal/Core/Entity/EntityDisplayBase.php)

πŸ‡¨πŸ‡ΏCzech Republic parisek

Working great thank you, moving to RTBC

πŸ‡¨πŸ‡ΏCzech Republic parisek

Added MR and disabled logging warning "Found multiple files for the", seems it's not wide issue, patch is enough :)

πŸ‡¨πŸ‡ΏCzech Republic parisek

Updated MR to support Drupal 10, unfortunately couldn't rebase

πŸ‡¨πŸ‡ΏCzech Republic parisek

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

πŸ‡¨πŸ‡ΏCzech Republic parisek

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

πŸ‡¨πŸ‡ΏCzech Republic parisek

Thank you for promoting this to a general solution. It's working great for me. RTBC.

πŸ‡¨πŸ‡ΏCzech Republic parisek

Thank you for the patch. It's working great for external URLs. I triggered new tests and everything is green.

Ready to be committed (RTBC).

πŸ‡¨πŸ‡ΏCzech Republic parisek

Thank you for your message. I'm just curious, why not switch all or most of the meta tag fields to text? I don't think it has any drawbacks.

πŸ‡¨πŸ‡ΏCzech Republic parisek

Created a simple patch to address this issue just for open graph images.

πŸ‡¨πŸ‡ΏCzech Republic parisek

Working great for me too. Tests are green. RTBC

πŸ‡¨πŸ‡ΏCzech Republic parisek

For me quickfix was add new permission for route entity.webform_submission.user via Custom Permissions β†’ module

πŸ‡¨πŸ‡ΏCzech Republic parisek

Added back to NR as I just found fatal error when some node translation is not published yet. It all goes down to this line

$node = $entity_repository->getTranslationFromContext($node);

We cannot pass upublished translation so I added check $node->isPublished() before

πŸ‡¨πŸ‡ΏCzech Republic parisek

For me this fixes disappeared language switcher links which introduced security fix for Drupal 9.5.5
https://www.drupal.org/project/drupal/issues/3348686 πŸ› [regression] Inaccessible language switcher links are removed before alternatives can be provided Fixed

Thank you

πŸ‡¨πŸ‡ΏCzech Republic parisek

Thank you, working for me, but CI test still not happy. Maybe best we can is separate problematic token to another issue?

πŸ‡¨πŸ‡ΏCzech Republic parisek

Created MR from attached patch and fixed #7

Changed to major as if both CC and BCC recepients are set, only later one is sent.

πŸ‡¨πŸ‡ΏCzech Republic parisek

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

Production build 0.69.0 2024