🇪🇸Spain @dima.iluschenko

Any updates there?

Hi @Martijn de Wit!
Do you know the approximate date when this will be ready?

dima.iluschenko → made their first commit to this issue’s fork.

dima.iluschenko → made their first commit to this issue’s fork.

Berdir → credited dima.iluschenko → .

dima.iluschenko → made their first commit to this issue’s fork.

Up :)

Hi Berdir,

I've updated the code. Pls, verify https://git.drupalcode.org/project/tmgmt/-/merge_requests/42/diffs.

Hello @Berdir,
could you please review this small patch?

Hello @Berdir,
I got your point. Thank you for the hint. I've adjusted PR https://git.drupalcode.org/project/tmgmt/-/merge_requests/42/diffs
If you need more help feel free to tag me.

Hi @Berdir
please see my latest changes - I have added logging for such cases: https://git.drupalcode.org/project/tmgmt/-/merge_requests/42.diff
Is there anything else needed?

@SteffenR Also we can't exclude only mask_icon in hook_tmgmt_translatable_fields_alter. We can exclude the entire meta tag field, so that won't help with my problem.

I agree with you, but I think in any case there should be a check for a string, since we cannot predict in which modules such a problem can still get out.

dima.iluschenko → created an issue.

Hi Steffen,

It's good that we can exclude fields from the translations. However, we shouldn't get php error and see a broken process during translation for contrib modules.

Attached the patch for skipping all except a string.

dima.iluschenko → created an issue.