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

Let field charts use the stream and time range of the current search #1938

Merged
merged 1 commit into from Mar 17, 2016

Conversation

@bernd
Copy link
Member

@bernd bernd commented Mar 16, 2016

Before this, stream and the time range settings have been taken from the
stored values in localStorage. This made the graphs stay the same even
if the user changed the time range options or searched in a different
stream.

Fixes #1785
Fixes graylog-labs/graylog2-web-interface#1620
Fixes graylog-labs/graylog2-web-interface#1618
Fixes graylog-labs/graylog2-web-interface#1485

Before this, stream and the time range settings have been taken from the
stored values in localStorage. This made the graphs stay the same even
if the user changed the time range options or searched in a different
stream.

Fixes #1785
Fixes graylog-labs/graylog2-web-interface#1620
Fixes graylog-labs/graylog2-web-interface#1618
Fixes graylog-labs/graylog2-web-interface#1485
@bernd bernd added this to the 2.0.0 milestone Mar 16, 2016
@edmundoa edmundoa self-assigned this Mar 17, 2016
@edmundoa
Copy link
Member

@edmundoa edmundoa commented Mar 17, 2016

LGTM 👍

edmundoa added a commit that referenced this pull request Mar 17, 2016
Let field charts use the stream and time range of the current search
@edmundoa edmundoa merged commit a9752c3 into master Mar 17, 2016
4 checks passed
4 checks passed
@garybot2
ci-server-integration Jenkins build graylog2-server-integration-pr 744 has succeeded
Details
@garybot2
ci-web-linter Jenkins build graylog-pr-linter-check 234 has succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@edmundoa edmundoa deleted the fix-issue-1785 branch Mar 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants