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

Delete FilterStateManager and QueryFilter :-D #59872

Merged
merged 8 commits into from Mar 12, 2020

Conversation

lizozom
Copy link
Contributor

@lizozom lizozom commented Mar 11, 2020

Summary

Delete FilterStateManager and QueryFilter

Dev Docs

Deleted unused legacy exports

  • Delete FilterStateManager
  • Delete QueryFilter
  • Delete SavedQuery re-export from legacy

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@lizozom lizozom requested a review from a team March 11, 2020 09:24
@lizozom lizozom requested a review from a team as a code owner March 11, 2020 09:24
@lizozom lizozom self-assigned this Mar 11, 2020
@lizozom lizozom added this to In progress in kibana-app-arch via automation Mar 11, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-arch (Team:AppArch)

@lizozom lizozom added the release_note:plugin_api_changes Contains a Plugin API changes section for the breaking plugin API changes section. label Mar 11, 2020
Copy link
Contributor

@flash1293 flash1293 left a comment

Choose a reason for hiding this comment

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

Kibana app changes look very good to me 🎉Another piece of the legacy platform gone!

Copy link
Contributor

@Dosant Dosant left a comment

Choose a reason for hiding this comment

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

LGTM!

@@ -5,9 +5,8 @@
*/

import { createSelector } from 'reselect';
Copy link
Contributor

Choose a reason for hiding this comment

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

Wonder why @elastic/siem isn't auto added to reviewers 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think they didn't define their code owners.
Think this would be a good idea!

kibana-app-arch automation moved this from In progress to Review in progress Mar 11, 2020
@lizozom lizozom requested a review from a team March 11, 2020 14:08
@lizozom
Copy link
Contributor Author

lizozom commented Mar 11, 2020

@elasticmachine merge upstream

@lizozom lizozom removed the request for review from a team March 11, 2020 14:14
@lizozom
Copy link
Contributor Author

lizozom commented Mar 11, 2020

@elasticmachine merge upstream

@lizozom
Copy link
Contributor Author

lizozom commented Mar 11, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

@lizozom lizozom merged commit c40e5ad into elastic:master Mar 12, 2020
kibana-app-arch automation moved this from Review in progress to Done in current release Mar 12, 2020
lizozom pushed a commit to lizozom/kibana that referenced this pull request Mar 12, 2020
* delete filter state manager and query filter :-D

* snapshot

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
lizozom pushed a commit that referenced this pull request Mar 12, 2020
* delete filter state manager and query filter :-D

* snapshot

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
gmmorris added a commit to gmmorris/kibana that referenced this pull request Mar 12, 2020
* master: (45 commits)
  skip flaky suite (elastic#59717)
  UI Metrics use findAll to retrieve all Saved Objects (elastic#59891)
  [Discover] Migrate Context mocha tests to use Jest (elastic#59658)
  [Maps] Move redux reducers and store logic to NP (elastic#58294)
  rebalance x-pack groups (elastic#58930)
  [Discover] Reimplement $route.reload when index pattern changes (elastic#59877)
  [Upgrade Assistant Meta] Breaking changes issue template (elastic#59745)
  Skip CI based on changes in PR (elastic#59939)
  [ML] Transforms: Replace KqlFilterBar with QueryStringInput. (elastic#59723)
  [ML] Functional tests - stabilize date_nanos test (elastic#59986)
  [ML] Typescripting client side endpoint functions (elastic#59928)
  a11y tests on adding columns to discover table (elastic#59375)
  fix graph plugin config path (elastic#59540)
  fix vega config issues (elastic#59737)
  [Upgrade Assistant] Open And Close Slight Refactor (elastic#59890)
  [ML] Adding shared services to ml setup contract (elastic#59730)
  [Visualize] Fix linked search behavior (elastic#59690)
  [ML] Register NP ML plugin for Kibana management section. (elastic#59762)
  [Lens] Adds using queries/filters for field existence endpoint (elastic#59033)
  Delete FilterStateManager and QueryFilter :-D (elastic#59872)
  ...
jkelastic pushed a commit to jkelastic/kibana that referenced this pull request Mar 12, 2020
* delete filter state manager and query filter :-D

* snapshot

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:New Platform release_note:plugin_api_changes Contains a Plugin API changes section for the breaking plugin API changes section. v7.7.0 v8.0.0
Projects
kibana-app-arch
  
Done in current release
Development

Successfully merging this pull request may close these issues.

None yet

5 participants