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] Enhances toast notifications to improve error reporting #173362
[ML] Enhances toast notifications to improve error reporting #173362
Conversation
Pinging @elastic/ml-ui (:ml) |
const { toasts } = notifications; | ||
toasts.addDanger( | ||
const toastNotificationService = toastNotificationServiceProvider(toasts); |
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.
const toastNotificationService = toastNotificationServiceProvider(toasts); | |
const toastNotificationService = useToastNotificationService(); |
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.
Done in 39a1915
@@ -30,6 +30,7 @@ import { parseUrlState } from '@kbn/ml-url-state'; | |||
|
|||
import { useMlKibana } from '../../../contexts/kibana'; | |||
import { isValidLabel, openCustomUrlWindow } from '../../../util/custom_url_utils'; | |||
import { toastNotificationServiceProvider } from '../../../services/toast_notification_service'; |
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.
import { toastNotificationServiceProvider } from '../../../services/toast_notification_service'; |
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.
Done in 39a1915
x-pack/plugins/ml/public/application/components/custom_urls/custom_urls.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/ml/public/application/components/rule_editor/rule_editor_flyout.js
Outdated
Show resolved
Hide resolved
0ff0ced
to
39a1915
Compare
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Async chunks
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: |
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!
Summary
Several enhancements to the error toast notifications in the ML plugin to improve error reporting.
The bulk of the changes are to add 'See the full error' buttons to the toasts allowing the user to see further details on the error that has occurred. Also makes minor edits to some of the error messages to improve clarity.
Also closes #171839 by changes to
x-pack/plugins/ml/public/application/jobs/jobs_list/components/utils.js
so that the the 'Jobs started successfully' toast is only shown if 1 or more jobs have been started successfully.Fixes #171839
Checklist