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

A few fixes required to get eui@8.x working in Kibana #1603

Merged
merged 4 commits into from
Feb 26, 2019

Conversation

jasonrhodes
Copy link
Member

Summary

  • EuiFilterButton was missing props in the shipped types (it was only using EuiEmptyButtonProps but it accepts about 5 extra props)

@thompsongl
Copy link
Contributor

@cchaos has you covered with this in #1589, as part of some design changes to these components.

@cchaos
Copy link
Contributor

cchaos commented Feb 26, 2019

Yeah, I saw that and fixed in #1589 as @thompsongl said. That should be going in soon.

@jasonrhodes
Copy link
Member Author

cchaos has you covered with this in #1589, as part of some design changes to these components.

@thompsongl thanks! I am going to leave this open temporarily while I am working through a ton of other errors I'm seeing in Kibana with eui@8.x. Not sure if they are Kibana errors or EUI errors or Jason errors yet.

@jasonrhodes jasonrhodes marked this pull request as ready for review February 26, 2019 17:56
Copy link
Contributor

@chandlerprall chandlerprall left a comment

Choose a reason for hiding this comment

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

Changes LGTM, needs a changelog entry

@@ -14,6 +14,7 @@ declare module '@elastic/eui' {
export interface EuiFilterButtonProps {
numFilters?: number;
numActiveFilters?: number;
hasActiveFilters?: boolean;
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks!

@jasonrhodes jasonrhodes merged commit 828dbd8 into elastic:master Feb 26, 2019
@jasonrhodes jasonrhodes deleted the fix-8.0-in-kibana branch February 26, 2019 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants