-
Notifications
You must be signed in to change notification settings - Fork 82
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: move localization keep patterns where they are needed #960
Conversation
b73b04a
to
ecaf713
Compare
ecaf713
to
6fcd8a4
Compare
|
The pattern /^subject.*/i is needed because in ICM versions lower than 7.10.38 the cost center admin is not allowed to read the list of users for the customer. In this case the translation key 'subject.has.no.permission.assigned' is used to show a corresponding error message on the cost center user assignment page. But the regular expression could be more specific |
So we can do it like this? 8f71726 |
6ca00fd
to
3b9b6d6
Compare
3b9b6d6
to
b8a340b
Compare
PR Type
[x] Refactoring (no functional changes, no API changes)
What Is the Current Behavior?
Localization keep patterns are managed in the cleanup script.
What Is the New Behavior?
Localization keep patterns are managed where they are defined. (with an annotation)
Does this PR Introduce a Breaking Change?
[ ] Yes
[x] No
Other Information
Hi @SGrueber @suschneider @MaxKless 😃
I think the
.error.
keep pattern in thescripts/clean-up-localizations.js
is outdated. I see that all of the error feedback regarding payments comes from the API, there should be no need to keep those translation keys in the PWA (also because we now fetch specific keys directly from the ICM). Regarding anything not payment related? I'm not sure...Please check if my assumption is correct.
In projects every uselessly translated localization is money 🤑
AB#72086