Skip to content

[7.8] [ML] Transforms: Fix API error message display for edit flyout. (#65494)#65695

Merged
walterra merged 1 commit intoelastic:7.8from
walterra:backport/7.8/pr-65494
May 8, 2020
Merged

[7.8] [ML] Transforms: Fix API error message display for edit flyout. (#65494)#65695
walterra merged 1 commit intoelastic:7.8from
walterra:backport/7.8/pr-65494

Conversation

@walterra
Copy link
Copy Markdown
Contributor

@walterra walterra commented May 7, 2020

Backports the following commits to 7.8:

…tic#65494)

Fixes an issue where the transform edit flyout would be hidden if an error occurred and the user closed the error toast. This fixes it by showing the error message within an callout in the flyout itself.
The bug is a side effect of the problem with the edit-button and it's corresponding React tree being within the transform list actions popover which will be solved in a follow up but possibly not for 7.8.0 which makes this workaround necessary.
@walterra walterra added backport This PR is a backport of another PR :ml labels May 7, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@walterra walterra self-assigned this May 7, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@walterra walterra merged commit fc9fc49 into elastic:7.8 May 8, 2020
@walterra walterra deleted the backport/7.8/pr-65494 branch May 8, 2020 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR :ml

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants