Skip to content

Re-introduce new filter bar - #29752

Merged
Bargs merged 6 commits into
elastic:masterfrom
Bargs:revertRevert25563
Jan 31, 2019
Merged

Bargs merged 6 commits into
elastic:masterfrom
Bargs:revertRevert25563

Conversation

@Bargs

@Bargs Bargs commented Jan 31, 2019

Copy link
Copy Markdown
Contributor

Summary

Fixes the build issues introduced in #25563 and re-introduces the new react/eui/typescript filter bar, essentially reverting the revert in #29662. I did have to resolve one merge conflict in query_bar.tsx, and re-deleted all of the old filter bar code where translation code had been added.

@Bargs Bargs added v7.0.0 Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// labels Jan 31, 2019
@Bargs
Bargs requested a review from a team as a code owner January 31, 2019 19:24
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-app

@elasticmachine

Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine

Copy link
Copy Markdown
Contributor

💔 Build Failed

Comment thread packages/kbn-es-query/package.json Outdated
@elasticmachine

Copy link
Copy Markdown
Contributor

💔 Build Failed

@Bargs
Bargs removed the request for review from cchaos January 31, 2019 22:50

@lukasolson lukasolson left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@elasticmachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@Bargs
Bargs merged commit 8a83955 into elastic:master Jan 31, 2019
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
Fixes the build issues introduced in elastic#25563 and re-introduces the new react/eui/typescript filter bar, essentially reverting the revert in elastic#29662. I did have to resolve one merge conflict in query_bar.tsx, and re-deleted all of the old filter bar code where translation code had been added.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v7.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants