-
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] Ensure chart interval is not smaller than bucket span #44895
[ML] Ensure chart interval is not smaller than bucket span #44895
Conversation
Pinging @elastic/ml-ui |
💚 Build Succeeded |
@@ -6,7 +6,7 @@ | |||
|
|||
import { SavedSearch } from 'src/legacy/core_plugins/kibana/public/discover/types'; | |||
import { IndexPattern } from 'ui/index_patterns'; | |||
import { parseInterval } from 'ui/utils/parse_interval'; | |||
import { parseInterval } from '../../../../../common/util/parse_interval'; |
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.
You can try path mapping
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.
I think a change like this would be outside the scope of this PR but is definitely something worth investigating to avoid these horrible relative paths.
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, including IE11, and LGTM
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 🍕
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
The interval used to draw each chart in the new job wizards should not be smaller than the bucket span.
When the bucket span changes, the chart intervals are now checked and updated if needed.
Also fixes issue where jobs created by a saved search should not be clonable through the wizards and therefore should not contain
created_by
meta data. This is how the old job wizards worked.