-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Translate attribute label with default translation helper function #13251
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
Conversation
Adds the translation function to customer attribute labels in Magento admin. This gives a chance to translate a label in the locale of a backend user.
@cmuench can you check the build please? It seems |
@miguelbalparda Yes, I will check this. |
@miguelbalparda I checked the failing test. It is indirectly caused by my change. |
@miguelbalparda I "fixed" the test not my code. Now the failing test runs in ~30ms instead of 25 seconds. My patch will speed up the complete test environment. |
This test tries to setup a test environment with two modules and one theme to test the generator. During the test run, the test environment and the complete module stack is processed by the generator. So the test takes on my machine 25 seconds with xdebug enabled. After resetting the registrar the complete test runs in ~50ms with xdebug enabled. Without the reset ~12.000 files are scanned. With the reset only 5 files are scanned. This test is not a unit test. It is an integration test.
Please don't insult other's people work. I've edited your comment. Thanks for the fix, we'll check it and report back. |
@miguelbalparda You should know me. I never insult any developers work. I only hate the written code. Every developer should hate the own written code and improve it. With this code I had some issues in a project. |
@magento-engcom-team Thanks, but the link is always redirecting me to https://github.com/magento/ |
Hello, I wanted to ask, whether this issue was fixed? Did I miss something? thx |
@magento-engcom-team is there a backport to 2.2 for this? |
Adds the translation function to customer attribute labels in Magento
admin. This gives a chance to translate a label in the locale of a
backend user.
Description
A custom attribute is created.
We have different frontend labels for the attribute. Store views with different locale settings exist.
Admin users are speaking different languages and need a localized attribute label.
This is not possible.
Fixed Issues (if relevant)
Manual testing scenarios
Contribution checklist