Skip to content

docs(search): document include_facets/include_histogram opt-in toggles#242

Open
steveatlc wants to merge 2 commits into
masterfrom
docs/search-include-toggles
Open

docs(search): document include_facets/include_histogram opt-in toggles#242
steveatlc wants to merge 2 commits into
masterfrom
docs/search-include-toggles

Conversation

@steveatlc
Copy link
Copy Markdown
Contributor

Summary

  • Document the two new optional search API body fields, include_facets and include_histogram, on the Query Console REST API examples page. Both default to false; callers that omit them now get an empty facets block and no histogram.
  • Update the Run an LCQL Query curl example to set both toggles to true so users see how to opt in.
  • The web Query Console always opts in (per web-app-frontend#3767), so the UI behavior is unchanged — the docs note this so REST/SDK/CLI users understand who is affected.

Reflects backend change in replay#442 and the matching webapp change in web-app-frontend#3767 (both merged 2026-05-22).

Test plan

  • mkdocs build — no new warnings introduced (the pre-existing 8-reference/story-tags.md → asset-tags.md warning is unrelated and present on master).
  • Audited rendered site/4-data-queries/index.html — admonition + curl tab render correctly with both new fields visible.
  • Reviewer sanity-check the wording against the API contract in replay#442 (ResolveIncludeToggle: nil → false).

🤖 Generated with Claude Code

The search API now accepts include_facets and include_histogram as
optional boolean request body fields. Both default to false: callers
that omit them receive an empty facets block and no histogram. Note
the new defaults in the Query Console docs and update the REST API
curl example to show how to opt in. Webapp behavior is unchanged
because the console sends explicit true for both.

Refs refractionPOINT/replay#442, refractionPOINT/web-app-frontend#3767

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

Broken Links Detected

The link checker found some issues:

* [ERROR] <file:///home/runner/work/documentation/documentation/site/8-reference/story-tags/asset-tags.md> | Cannot find file: File not found. Check if file exists and path is correct

Please fix these broken links before merging.

Full results

See the uploaded artifact for complete link check results.

@steveatlc steveatlc requested a review from tomaz-lc May 26, 2026 17:14
The link `[lc:asset:*](asset-tags.md)` resolved into 8-reference/
where no asset-tags.md exists; the canonical file lives at
docs/2-sensors-deployment/asset-tags.md and is already referenced
correctly later in the same page. Use the same relative path.

Unblocks the docs link checker on PR #242.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

1 participant