silverstripe / silverstripe-admin

Silverstripe Admin Component
BSD 3-Clause "New" or "Revised" License
25 stars 91 forks source link

Untranslatable js text `Search options` #1689

Closed pschiffe closed 4 months ago

pschiffe commented 4 months ago

Module version(s) affected

2.1.17

Description

Hello, there is an untranslatable text Search options in js file SearchBox.js: https://github.com/silverstripe/silverstripe-admin/blob/50751546b57f32b66f8a173d64daa03d017d80ae/client/src/components/Search/SearchBox.js#L292C35-L292C49

Also, some texts are not in transifex.com, such as Admin.SEARCH or Admin.ADVANCED.

Cheers

How to reproduce

Check source code.

Possible Solution

No response

Additional Context

No response

Validations

PRs

pschiffe commented 4 months ago

There's also this text I'm not able to translate or find out where it's coming from:

Screenshot 2024-02-24 at 20-20-03 Jedáleň - Správa jedál

Locally I see it only in the file vendor/silverstripe/admin/client/dist/js/vendor.js

GuySartorelli commented 4 months ago

there is an untranslatable text Search options in js file SearchBox.js

Thanks for reporting this. Would you like to submit a pull request to fix this? There are examples in that same file of how to make the string localisable. If it affects CMS 4 the PR should target the 1.13 branch, otherwise target the 2.1 branch.

Also, some texts are not in transifex.com, such as Admin.SEARCH or Admin.ADVANCED.

Please open a separate issue for this - the root cause is completely different and so needs to be handled separately.

There's also this text I'm not able to translate or find out where it's coming from

This is coming from a third-party package. Please create a separate issue about this, including more details such as what type of field that is, and the PHP code used to initialise that field.

GuySartorelli commented 4 months ago

PR merged. This will be automatically tagged by GitHub actions.