-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[i18n] Translate ML - items grid and item select #28236
[i18n] Translate ML - items grid and item select #28236
Conversation
second merge
Pinging @elastic/ml-ui |
💔 Build Failed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@peteharverson code LGTM but could not open tabs on UI to check that nothing is broken.
Also @peteharverson we could not find where mlItemSelect directive is used(it looks like nowhere)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and LGTM.
@pavel06081991 you're right /ml/public/components/item_select
is not used any more. Once this is merged I will delete it in another PR to keep the work separate from the localization changes.
💚 Build Succeeded |
* Translate item_select * Translate items_grid
Translate ml -> public -> components -> items_grid
Translate ml -> public -> components -> item_select
Issue: #25454
Guidline
Readme