Skip to content

[7.x] [ML] Transforms: Fix API error message display for edit flyout. (#65494)#65694

Merged
walterra merged 1 commit intoelastic:7.xfrom
walterra:backport/7.x/pr-65494
May 8, 2020
Merged

[7.x] [ML] Transforms: Fix API error message display for edit flyout. (#65494)#65694
walterra merged 1 commit intoelastic:7.xfrom
walterra:backport/7.x/pr-65494

Conversation

@walterra
Copy link
Copy Markdown
Contributor

@walterra walterra commented May 7, 2020

Backports the following commits to 7.x:

…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 the backport This PR is a backport of another PR label May 7, 2020
@walterra walterra self-assigned this May 7, 2020
@walterra walterra added the :ml label May 7, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@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 7860777 into elastic:7.x May 8, 2020
@walterra walterra deleted the backport/7.x/pr-65494 branch May 8, 2020 06:34
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