-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[ML] Adding ability to override number of sample lines in File Data Visualizer #29214
[ML] Adding ability to override number of sample lines in File Data Visualizer #29214
Conversation
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.
LGTM
Code LGTM - I'll give it a play around tomorrow but didn't want to be a blocker 😄 |
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.
LGTM
@@ -2,6 +2,28 @@ | |||
|
|||
exports[`Overrides render overrides 1`] = ` | |||
<EuiForm> | |||
<EuiFormRow | |||
describedByIds={Array []} | |||
error="Value must be greater than 3 and less than or equal to 1000000" |
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.
Isn't it possible to localize this error message?
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.
in the snapshot file? i don't know?
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.
Looking in other snapshots, seems only the label
is localized, and not error
text,
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.
Doesn't look like it based on already translated snapshot files:
Line 32 in 735cc82
"Use lowercase alphanumerics (a-z and 0-9), hyphens or underscores; must start and end with an alphanumeric character", |
💚 Build Succeeded |
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.
LGTM
User can now set a custom value between
3
and100000
for the number of lines thefind_file_structure
will use when analysing the file.Form validation has been added which required refactoring of the overrides component.
![image](https://user-images.githubusercontent.com/22172091/51637020-8de6c000-1f52-11e9-8683-8ed5fb71a492.png)
Fixes #25458