Skip to content

add timefilter to search source instead of inheriting from root search source - #16159

Merged
nreese merged 1 commit into
elastic:masterfrom
nreese:controls_search_source
Jan 19, 2018
Merged

nreese merged 1 commit into
elastic:masterfrom
nreese:controls_search_source

Conversation

@nreese

@nreese nreese commented Jan 19, 2018

Copy link
Copy Markdown
Contributor

#15852 added use time filter option to input controls.

The implementation is incorrect in PR 15852. It adds the timefilter by allowing the search source to inherit from the root search source. This introduced a bug in that the root search source also included pinned filters and the intent of the feature is only filter by time and not by pinned filters.

This PR fixes the implementation to avoid inheriting from root search source and instead just add the time filter directly to the search source.

@nreese nreese added :Sharing Feature:Input Control Input controls visualization labels Jan 19, 2018
@nreese
nreese requested a review from kobelb January 19, 2018 16:26

@kobelb kobelb 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

@nreese
nreese merged commit caa38d0 into elastic:master Jan 19, 2018
nreese added a commit to nreese/kibana that referenced this pull request Jan 19, 2018
nreese added a commit that referenced this pull request Jan 19, 2018
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Input Control Input controls visualization

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants