- Issue created by @poker10
- Status changed to Needs work
over 1 year ago 8:29pm 28 August 2023 - 🇸🇰Slovakia poker10
Something like this should fix the issue (based on D7 patch). But tests are still needed I think (so keeping at NW).
- last update
over 1 year ago 30,060 pass - Status changed to Needs review
over 1 year ago 7:03pm 29 August 2023 - last update
over 1 year ago 30,063 pass - last update
over 1 year ago 30,057 pass, 1 fail - 🇸🇰Slovakia poker10
Added a test for checking the custom string in the generated JS file. Uploading also test-only patch, which should hopefully fail.
The last submitted patch, 4: 3383888-4_test-only.patch, failed testing. View results →
- Status changed to RTBC
over 1 year ago 1:53am 30 August 2023 - 🇺🇸United States smustgrave
Thanks for including a test-only patch to show this is an issue!
Looking at the actual fix I don't see anything standing out that could be a problem.
Issue summary appears complete
Going to mark this.
- last update
over 1 year ago 30,124 pass, 1 fail The last submitted patch, 4: 3383888-4_test-only.patch, failed testing. View results →
- last update
over 1 year ago 30,129 pass, 1 fail The last submitted patch, 4: 3383888-4_test-only.patch, failed testing. View results →
- last update
over 1 year ago 30,130 pass, 1 fail The last submitted patch, 4: 3383888-4_test-only.patch, failed testing. View results →
- last update
over 1 year ago 30,130 pass, 1 fail The last submitted patch, 4: 3383888-4_test-only.patch, failed testing. View results →
- last update
over 1 year ago 30,140 pass, 1 fail The last submitted patch, 4: 3383888-4_test-only.patch, failed testing. View results →
32:10 28:00 RunningThe last submitted patch, 4: 3383888-4_test-only.patch, failed testing. View results →
-
larowlan →
committed a977482f on 10.1.x
Issue #3383888 by poker10: Drupal.t() does not respect...
-
larowlan →
committed a977482f on 10.1.x
- Status changed to Fixed
over 1 year ago 5:41am 11 September 2023 -
larowlan →
committed 0e6eec07 on 11.x
Issue #3383888 by poker10: Drupal.t() does not respect...
-
larowlan →
committed 0e6eec07 on 11.x
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
Tested this locally with umami install profile.
Unfortunately the steps to reproduce don't work anymore because of changes to field UI.
But you can still see it with these, so updating the issue summary:
$settings['locale_custom_strings_es'][''] = [ 'Label' => 'LabelCUSTOM', 'Edit' => 'EditCUSTOM', 'List additional actions' => 'Additional actions ACTIONS!', ];
Committed and pushed to 11.x
Backported to 10.0.xThanks folks
Automatically closed - issue fixed for 2 weeks with no activity.