Skip to content

[ML] Transforms: Fix error toasts.#48509

Merged
walterra merged 3 commits intoelastic:masterfrom
walterra:transform-fix-error-toasts
Oct 18, 2019
Merged

[ML] Transforms: Fix error toasts.#48509
walterra merged 3 commits intoelastic:masterfrom
walterra:transform-fix-error-toasts

Conversation

@walterra
Copy link
Copy Markdown
Contributor

@walterra walterra commented Oct 17, 2019

Summary

Fixes #46434.

Previously, error messages were just output within the toast as raw text which could result in overflows and overall unreadable results:

image

This update fixes it by providing a "View details" button which opens a modal with the properly formatted error message. Plain text error message up to 140 character will still be shown in the toast itself.

data-frame-transform-error-modal

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

For maintainers

@walterra walterra added bug Fixes for quality problems that affect the customer experience :ml v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Feature:Transforms Transforms v7.5.0 v7.6.0 labels Oct 17, 2019
@walterra walterra requested a review from a team as a code owner October 17, 2019 11:22
@walterra walterra self-assigned this Oct 17, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

Copy link
Copy Markdown
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@walterra walterra changed the title [transform] Fix error toasts. [ML] Transforms: Fix error toasts. Oct 17, 2019
@walterra
Copy link
Copy Markdown
Contributor Author

retest

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@walterra
Copy link
Copy Markdown
Contributor Author

retest

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Copy link
Copy Markdown
Contributor

@darnautov darnautov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@walterra walterra merged commit 39c8aeb into elastic:master Oct 18, 2019
@walterra walterra deleted the transform-fix-error-toasts branch October 18, 2019 08:11
walterra added a commit to walterra/kibana that referenced this pull request Oct 18, 2019
Previously, error messages were just output within the toast as raw text which could result in overflows and overall unreadable results. This update fixes it by providing a "View details" button which opens a modal with the properly formatted error message. Plain text error message up to 140 character will still be shown in the toast itself.
walterra added a commit to walterra/kibana that referenced this pull request Oct 18, 2019
Previously, error messages were just output within the toast as raw text which could result in overflows and overall unreadable results. This update fixes it by providing a "View details" button which opens a modal with the properly formatted error message. Plain text error message up to 140 character will still be shown in the toast itself.
@sophiec20
Copy link
Copy Markdown
Contributor

Is there anyway we can get more info into the toast message... for example security_exception ?

It's a pain to have to open the toast message to see more, especially if there are more than one.

walterra added a commit that referenced this pull request Oct 18, 2019
Previously, error messages were just output within the toast as raw text which could result in overflows and overall unreadable results. This update fixes it by providing a "View details" button which opens a modal with the properly formatted error message. Plain text error message up to 140 character will still be shown in the toast itself.
walterra added a commit that referenced this pull request Oct 18, 2019
Previously, error messages were just output within the toast as raw text which could result in overflows and overall unreadable results. This update fixes it by providing a "View details" button which opens a modal with the properly formatted error message. Plain text error message up to 140 character will still be shown in the toast itself.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Fixes for quality problems that affect the customer experience Feature:Transforms Transforms :ml release_note:skip Skip the PR/issue when compiling release notes v7.5.0 v7.6.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ML] Transform toast errors should be easier to read

5 participants