Skip to content

[Lens] Add date histogram interval to column name#48271

Merged
chrisdavies merged 10 commits intoelastic:masterfrom
chrisdavies:lens/auto-date
Oct 16, 2019
Merged

[Lens] Add date histogram interval to column name#48271
chrisdavies merged 10 commits intoelastic:masterfrom
chrisdavies:lens/auto-date

Conversation

@chrisdavies
Copy link
Contributor

If a date histogram is selected, we now will display the per whatever interval suffix that esaggs assigns:

image

@chrisdavies chrisdavies added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Feature:Lens v7.5.0 labels Oct 15, 2019
@chrisdavies chrisdavies requested review from a team and wylieconlon October 15, 2019 19:02
Copy link
Contributor

@wylieconlon wylieconlon left a comment

Choose a reason for hiding this comment

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

This change LGTM. Tested in app and dashboard.

@elasticmachine
Copy link
Contributor

💔 Build Failed

@chrisdavies
Copy link
Contributor Author

retest

@wylieconlon wylieconlon added the Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// label Oct 15, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed


return `${expression} | ${filterRatios
.map(([id]) => `lens_calculate_filter_ratio id=${id}`)
.join(' | ')}`;
Copy link
Contributor

Choose a reason for hiding this comment

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

This filter ratio part is no longer needed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Lens release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v7.5.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants