Skip to content
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

[Cases] Refactor plugin cases "Attach to existing case" (select case) modal logic out of the timelines plugin #125782

Closed
3 tasks
Tracked by #123183
academo opened this issue Feb 16, 2022 · 2 comments · Fixed by #126071
Closed
3 tasks
Tracked by #123183
Assignees
Labels
Feature:Cases Cases feature Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.2.0

Comments

@academo
Copy link
Contributor

academo commented Feb 16, 2022

See epic #123183

The existing integration between alert timelines and cases consists of a series of hooks, contexts and callbacks that are not entirely reusable for other plugins and that require the timelines plugin to keep logic to track of open flyouts and to update cases to attach information to them.

To detach the "Attach to existing case" from timelines some code refactor is required in the display and hide the modal logic.

Currently the timelines plugin has the logic to open and close the cases modal, this logic should be moved to the cases plugin control (see EPIC).

Acceptance criteria:

  • The timelines plugin doesn't handle opening and closing the select cases modal
  • Tests are updated in the timelines and cases plugin
  • Old code is kept for backwards compatibility (to be removed in following issues)
@academo academo added Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) Feature:Cases Cases feature v8.2.0 labels Feb 16, 2022
@academo academo self-assigned this Feb 16, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops-cases (Feature:Cases)

@academo academo changed the title [Cases] Refactor plugin cases "Attach to existing" (select case) modal logic out of the timelines plugin [Cases] Refactor plugin cases "Attach to existing case" (select case) modal logic out of the timelines plugin Feb 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Cases Cases feature Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.2.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants