Skip to content

Conversation

@stephmilovic
Copy link
Contributor

@stephmilovic stephmilovic commented Apr 15, 2020

Summary

This PR fixes the following bugs

  1. Resolves https://github.com/elastic/siem-team/issues/616 The timeline id of timeline-1 was being used to generate the timeline url when creating a new case from timeline rather than the correct timeline saved object id. Fixed!
  2. Resolves https://github.com/elastic/siem-team/issues/617 "Reporter" and "Tags" need to be refetched when all the other table gets refetched
  3. When debugging https://github.com/elastic/siem-team/issues/617, I found another bug where only the cases data was getting refetched when filterOptions and queryParams updated. I added a useEffect to update the other data (case status header and reporters/tags)

Checklist

Delete any items that are not applicable to this PR.

@stephmilovic stephmilovic requested a review from a team as a code owner April 15, 2020 20:20
@stephmilovic stephmilovic added release_note:skip Skip the PR/issue when compiling release notes Team:SIEM v7.7.0 v7.8.0 v8.0.0 labels Apr 15, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/siem (Team:SIEM)

@stephmilovic stephmilovic changed the title [SIEM] [Cases] Insert timeline and reporters/tags in table bug fixes [SIEM] [Cases] [skip-ci] Insert timeline and reporters/tags in table bug fixes Apr 15, 2020
@stephmilovic stephmilovic self-assigned this Apr 15, 2020
@stephmilovic stephmilovic changed the title [SIEM] [Cases] [skip-ci] Insert timeline and reporters/tags in table bug fixes [SIEM] [Cases] Insert timeline and reporters/tags in table bug fixes Apr 15, 2020
Copy link
Contributor

@XavierM XavierM left a comment

Choose a reason for hiding this comment

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

LGTM Tested/ Pair review with @stephmilovic
Thank you for adding the test with the bugs fixes
Really nice!!!

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

@stephmilovic stephmilovic merged commit 31ed266 into elastic:master Apr 16, 2020
@stephmilovic stephmilovic deleted the case-bugfixing branch April 16, 2020 06:53
XavierM pushed a commit that referenced this pull request Apr 16, 2020
…fixes (#63642) (#63658)

* [SIEM] [Cases] Insert timeline and reporters/tags in table bug fixes (#63642)

* fix type err
gmmorris added a commit to gmmorris/kibana that referenced this pull request Apr 17, 2020
* master: (56 commits)
  [i18n] Update CODEOWNERS (elastic#63354)
  add platform team definition of done (elastic#59993)
  [SIEM] move away from Joi for importing/exporting timeline (elastic#62125)
  Fix discover preserve url (elastic#63580)
  [alerting] Adds an alertServices mock and uses it in siem, monitoring and uptime (elastic#63489)
  Closes elastic#63109 for Service Map by resetting edges styles for the selected node (elastic#63655)
  MIgrated index_header to react (elastic#63490)
  Index pattern management UI -> TypeScript and New Platform Ready (indexed_fields_table) (elastic#63364)
  [SIEM] [Cases] Insert timeline and reporters/tags in table bug fixes (elastic#63642)
  [Reporting] Make usable default element positions (elastic#63191)
  [Reporting] Switch Serverside Config Wrapper to NP (elastic#62500)
  [Reporting] Add "warning" status as an alternate type of completed job (elastic#63498)
  Split action types into own page (elastic#63516)
  [Lens] Only show copy on save for previously saved docs (elastic#63535)
  Update README.md (elastic#63622)
  Bugfix clear saved query crashes kibana on Discover in some cases (elastic#63554)
  Add uptime CODEOWNER entries. (elastic#63616)
  [ML] Extract apiDoc params from the schema definitions (elastic#62933)
  Fix alerting documentation encryption key requirement (elastic#63512)
  Fix CODEOWNERS and sass lint paths (elastic#63552)
  ...
@MindyRS MindyRS added the Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. label Sep 23, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-solution (Team: SecuritySolution)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:SIEM v7.7.0 v7.8.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants