πŸ‡ΊπŸ‡¦Ukraine @Vadym.Tseiko

Account created on 14 September 2021, over 3 years ago
#

Merge Requests

More

Recent comments

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Hey guys, have a problem with usage of copy widget right after clone of LB with cloned inline blocks, so basically inline blocks created but before second save right after clone, CM's can try to add translation to the node with content, and basically blocks revision and its id is null at that point, what results in error. I fixed this by modifying patch for clone that saves blocks right after createDuplicate(), but still revision is null. So I had also to add 2 lines of code to patch#7 in order to fix this issue for this particular case.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Eventually to remove sample content generation I had to override SampleValueGenerator service methods, but it would be nice if we can have some sort of disable sample value config.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

I meant that there is no way(I haven't found) to prevent sample content generation, and it would be nice feature to add an option(maybe globally, maybe on field(instance) level) that leave all fields in default way with default values if set if not - empty.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Hi @silvi.addweb, I have updated from 2.1.2 to 3.0.0 and this problem occurred. Field inside layout builder component. Maybe this is the problem?

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Sorry applied wrong version, that just came out.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

I added patch made from MR. Fix works for me - Drupal 10.2.6

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Set to Needs review to get clarification, what further needs to be done.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Hi @jwilson3, I thought that in comment #3 you meant the checkbox in formatted settings. When I re-rolled the patch I added to the formatter settings checkbox, labeled as 'Uploaded dimensions', if such enabled - then this code fires.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Forgot to mention - we use the gin admin theme version 3.0.0-rc9. Also added a patch that adds a scrollbar.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Re-roll patch #18, so it can be comaptible with version 1.18.0

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

I found that this happends with blocks with long content, like sliders(view-block styled as slider) etc.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Drupal 10.2.4, PHP 8.1 - Could not apply patch after update drupal/custom_breadcrumbs 1.1.0 => 1.1.1.
Re-roll patch:

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

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

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Having issue on D10.1.6 updated from D9.5.11 when change ckeditor to ckeditor5. When adding anchor link button in text and formats, any of the editor fields with the format where this plugin used, had js error that plugin does not exists and breaks the toolbar of the editor.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Applied local patch made from interdiff #14 - worked fine for me. Thanks guys.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Hi everyone, I have a similar problem, my project is configured via detection and selection URL by domains method and I need to switch language to the correct domain accordingly to the user's browser preferences. But in my case, when using detection and selection via browser I'm constantly switched to my preferred language without the ability to switch language only once(what I'm achieved with custom event subscriber and redirect). I added a custom code that redirects user but now I'm having a problem with SEO as Google search bots is redirected too, and all machine services return 302 status code besides the basic English one as those bots I guess have English as a preference. Is there a way for redirect to not mess with bots or to detect them and not redirect if that not human-users, none of those patches fixed behavior when the user is constantly redirected and none of them set the right domain in the URL as a result I getting always URL from with browser switch to the preferred language? Has anyone solved this problem somehow?

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Tried: drush updb -y, re-install the module, check updates, caches cleared. I think there need to be added some hook_entity_update or base_field_definition.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

I seek further and found that if try to create view with filter on some of fields that connected to recently read module error appears that no such database table. This problem appears after update of the module.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Hello guys, I have a similar problem here after the module update.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Checked on my dev environment - works as well, thx guys. Cheers.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Faced the same problem on my project.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Hi @darvanen, the user's email is optional because most of the flow with content management is inside the project. I'm agreed with you that better to use the email address and this is not the best flow, but I've been asked, to fix a problem of creating users with no email. And yes first I've added changes to the user Constraint too. But then I thought that maybe there can be other fields that have optional email(like forms with not required field email, leave contact information or something so I added this check to the advanced email validator) So if field is required it would not let to leave field be empty then. But I guess your approach is better. Anyway, thanks for your response I will replace my patch to patch #6.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

I solved this issue with a small local patch, if someone will get in the same spot you can use it.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

Hi @vbouchet, I have the same issue here. My project is drupal/core-recommended: 9.5.7. I use drupal/layout_builder_iframe_modal: 1.3.0. At the moment we faced this issue, after adding multilingual functionality via drupal core built-in multilingual functionality. Using language detection via URL by domain. Just as Mark mentioned I tried to change following: 'destination' => Url::fromRoute('layout_builder_iframe_modal.redirect')->toString(TRUE)->getGeneratedUrl(), back to 'destination' => '/layout_builder_iframe_modal/redirect', and it works fine in that case. When I work with iframe without multilingual changes everything works fine.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

It seems that my commit with the updated phpcs fix did not add. Need to re-add changes to the repository.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

According to maintainers comment, branch 8.x-2.x-dev - only bugfix. Move update for fix reported by phpcs to branch 3.0.x-dev.

πŸ‡ΊπŸ‡¦Ukraine Vadym.Tseiko

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

Production build 0.71.5 2024