Skip to content

Document metrics grid aggregations in Discover - #8204

Merged
miguel-sanchez-elastic merged 11 commits into
mainfrom
document-discover-metrics-aggregations
Sep 22, 2026
Merged

miguel-sanchez-elastic merged 11 commits into
mainfrom
document-discover-metrics-aggregations

Conversation

@miguel-sanchez-elastic

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • Check the docs preview for the aggregations subsection
  • Confirm inline applies_to badges (Serverless, Stack 9.6+ / Planned)

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Elastic Docs AI PR menu

Check the box to run an AI review for this pull request.

  • Review docs changes (docs-review). Status: not started.

Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team.

@github-actions

github-actions Bot commented Sep 4, 2026 •

Copy link
Copy Markdown
Contributor

🔍 Preview links for changed docs

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

✅ Elastic Docs Style Checker (Vale)

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale.

@kpatticha kpatticha left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

lgtm

@miguel-sanchez-elastic
miguel-sanchez-elastic marked this pull request as ready for review September 7, 2026 16:46
@miguel-sanchez-elastic
miguel-sanchez-elastic requested a review from a team as a code owner September 7, 2026 16:46

@mdbirnstiehl mdbirnstiehl left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

One comment, but otherwise LGTM.

Comment thread solutions/observability/infra-and-hosts/discover-metrics.md Outdated
Co-authored-by: Mike Birnstiehl <114418652+mdbirnstiehl@users.noreply.github.com>
@miguel-sanchez-elastic

miguel-sanchez-elastic commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor Author

One comment, but otherwise LGTM.

Hi @mdbirnstiehl , I have applied the suggested change.

Let me know if the screenshot shows it in the way you expect it:
image

@mdbirnstiehl

Copy link
Copy Markdown
Member

One comment, but otherwise LGTM.

Hi @mdbirnstiehl , I have applied the suggested change.

Let me know if the screenshot shows it in the way you expect it: image

LGTM!

@miguel-sanchez-elastic

Copy link
Copy Markdown
Contributor Author

Thank you @mdbirnstiehl !

We'll merge it once we deploy our changes on elastic/kibana#288597

@miguel-sanchez-elastic
miguel-sanchez-elastic added this pull request to the merge queue Sep 22, 2026
Merged via the queue into main with commit b9bce8c Sep 22, 2026
12 checks passed
@miguel-sanchez-elastic
miguel-sanchez-elastic deleted the document-discover-metrics-aggregations branch September 22, 2026 08:32
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.

3 participants