Skip to content

Clear changes when canceling an edit to an alert#60518

Merged
mikecote merged 4 commits intoelastic:masterfrom
mikecote:alerting/clear-edit-flyout-on-cancel
Mar 19, 2020
Merged

Clear changes when canceling an edit to an alert#60518
mikecote merged 4 commits intoelastic:masterfrom
mikecote:alerting/clear-edit-flyout-on-cancel

Conversation

@mikecote
Copy link
Copy Markdown
Contributor

@mikecote mikecote commented Mar 18, 2020

Fixes #60328

In this PR, I'm fixing a bug where the edit alert flyout wouldn't reset the values after canceling.

@mikecote mikecote added Feature:Alerting v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.7.0 Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// labels Mar 18, 2020
@mikecote mikecote requested a review from a team as a code owner March 18, 2020 14:41
@mikecote mikecote self-assigned this Mar 18, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Mar 18, 2020

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

YulNaumenko
YulNaumenko approved these changes Mar 18, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@YulNaumenko YulNaumenko left a comment

Choose a reason for hiding this comment

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

LGTM

pmuellr
pmuellr approved these changes Mar 18, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@pmuellr pmuellr left a comment

Choose a reason for hiding this comment

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

LGTM

@kibanamachine
Copy link
Copy Markdown
Contributor

kibanamachine commented Mar 19, 2020

💛 Build succeeded, but was flaky

Test Failures

Kibana Pipeline / kibana-xpack-agent / adds a field to the timeline when the user drags and drops a field.Fields Browser Editing the timeline adds a field to the timeline when the user drags and drops a field

Link to Jenkins

Standard Out

Failed Tests Reporter:
  - Test has failed 6 times on tracked branches: https://github.com/elastic/kibana/issues/60023

Stack Trace

CypressError: Timed out retrying: Expected to find element: '[data-test-subj="header-text-host.geo.country_name"]', but never found it.
    at cypressErr (http://elastic:changeme@localhost:61121/__cypress/runner/cypress_runner.js:149934:9)
    at throwErr (http://elastic:changeme@localhost:61121/__cypress/runner/cypress_runner.js:149867:11)
    at Object.throwErrByPath (http://elastic:changeme@localhost:61121/__cypress/runner/cypress_runner.js:149915:3)
    at retry (http://elastic:changeme@localhost:61121/__cypress/runner/cypress_runner.js:144197:19)
    at onFailFn (http://elastic:changeme@localhost:61121/__cypress/runner/cypress_runner.js:132448:16)
    at tryCatcher (http://elastic:changeme@localhost:61121/__cypress/runner/cypress_runner.js:167415:23)
    at Promise._settlePromiseFromHandler (http://elastic:changeme@localhost:61121/__cypress/runner/cypress_runner.js:165351:31)
    at Promise._settlePromise (http://elastic:changeme@localhost:61121/__cypress/runner/cypress_runner.js:165408:18)
    at Promise._settlePromise0 (http://elastic:changeme@localhost:61121/__cypress/runner/cypress_runner.js:165453:10)
    at Promise._settlePromises (http://elastic:changeme@localhost:61121/__cypress/runner/cypress_runner.js:165528:18)
    at Async../node_modules/bluebird/js/release/async.js.Async._drainQueue (http://elastic:changeme@localhost:61121/__cypress/runner/cypress_runner.js:162140:16)
    at Async../node_modules/bluebird/js/release/async.js.Async._drainQueues (http://elastic:changeme@localhost:61121/__cypress/runner/cypress_runner.js:162150:10)
    at Async.drainQueues (http://elastic:changeme@localhost:61121/__cypress/runner/cypress_runner.js:162024:14)

History

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

@mikecote mikecote merged commit 05a0625 into elastic:master Mar 19, 2020
mikecote added a commit to mikecote/kibana that referenced this pull request Mar 19, 2020
@mikecote
Clear changes when canceling an edit to an alert (elastic#60518)
6a9a72c 
* Clear alerting edit flyout after canceling an edit

* Add functional test

* Fix merge conflicts
@mikecote mikecote mentioned this pull request Mar 19, 2020
Merged
mikecote added a commit that referenced this pull request Mar 19, 2020
@mikecote
Clear changes when canceling an edit to an alert (#60518) (#60634)
8bb1ae8 
* Clear alerting edit flyout after canceling an edit

* Add functional test

* Fix merge conflicts
gmmorris added a commit to gmmorris/kibana that referenced this pull request Mar 20, 2020
@gmmorris
Merge branch 'master' into alerting/tls-warning
becff42 
* master:
  [ML] Use a new ML endpoint to estimate a model memory (elastic#60376)
  [Logs UI] Correctly update the expanded log rate table rows (elastic#60306)
  fixes drag and drop flakiness (elastic#60625)
  Removing isEmptyState from embeddable input (elastic#60511)
  [Cross Cluster Replication] NP Shim (elastic#60121)
  Clear changes when canceling an edit to an alert (elastic#60518)
  Update workflow syntax (elastic#60626)
  Updating project assigner workflows to v2.0.0 of the action and back to default tokens (elastic#60577)
  migrate saved objects management edition view to react/typescript/eui (elastic#59490)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pmuellr pmuellr pmuellr approved these changes

@YulNaumenko YulNaumenko YulNaumenko approved these changes

Assignees

@mikecote mikecote

Labels

Feature:Alerting release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v7.7.0 v8.0.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Alert appears to keep values after cancelling an edit

5 participants

@mikecote @elasticmachine @kibanamachine @pmuellr @YulNaumenko