-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Vis] Move propFilter from ui/filter to ui/agg_types/filter #36875
Conversation
💔 Build Failed |
💚 Build Succeeded |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I pushed a small change using proper generics for the filter function so no types are lost when using it. Besides that the code lgtm if you find time and it's easy enough to convert the test to jest I would appreciate that, but otherwise it looks good to me.
💚 Build Succeeded |
This comment has been minimized.
This comment has been minimized.
💚 Build Succeeded |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still LGTM
…36875) * Move propFilter from ui/filter to ui/agg_types/filter * Return default value * Make propFilter generic * Convert propFilter from a mocha test to a jest test
Summary
Move propFilter from ui/filter to ui/agg_types/filter
Checklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.For maintainers