Update CPS placeholder pricing - #7552
Conversation
Elastic Docs AI PR menuCheck the box to run an AI review for this pull request.
Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team. |
✅ 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. |
eedugon
left a comment
There was a problem hiding this comment.
LGTM!
Next time when modifying a snippet please include the preview link of at least one of the pages that use the snippet, as otherwise is difficult to find out :)
yetanothertw
left a comment
There was a problem hiding this comment.
Thanks for the ping! I'll make sure to pull these changes into my branch.
Left a tiny comment for simplification. 🌻
Co-authored-by: Vlada Chirmicci <vlada.chirmicci@elastic.co>
There was a problem hiding this comment.
Docs review summary
Focus areas
- Style and clarity: Vale flagged only a pre-existing
executeword-choice suggestion (unrelated to this diff, line untouched) and a routinemapped_pageswarning (metadata unchanged by this PR). One inline suggestion posted ondeploy-manage/_snippets/cps-billing.mdfor a hard-to-parse sentence introduced by the rewrite. - Jargon: No unexplained Elastic-internal jargon introduced.
{{cps}}/{{cps-cap}}/{{cps-init}}substitutions expand correctly and CPS is spelled out on first use at each transclusion site (cross-project-search-config.md,serverless-project-billing-dimensions.md). - Frontmatter and applies_to: No frontmatter changed by this PR. The
applies_to: serverless: previewblocks that gated the now-removed per-product CPS sections were deleted along with those sections — no orphanedapplies_toblocks remain. - Content type fit: No mismatch. This is a reference/how-to billing page and the changes are in-place prose edits, consistent with the existing structure.
- Parent issue satisfaction: Satisfied. The PR fixes
elastic/docs-content-internal#1500by removing project-type-specific CPS billing language and replacing it with a single unified snippet, per the pricing model change described in the PR body.
Notes
- Verified no dangling references remain to the removed anchors (
observability-billing-cps,security-billing-cps) or the deleted snippet_snippets/cps-billing-obs-sec.md— repo-wide search found none. - The consolidated
_snippets/cps-billing.mdis transcluded from two other pages (cross-project-search-config.mdandserverless-project-billing-dimensions.md), and both read correctly with the new unified content.
Generated by Docs review agent for #7552 · sonnet 162.8 AIC · ⌖ 9.04 AIC · ⊞ 866
Summary
Because pricing approach no longer varies by origin project type, I removed the project type-specific info on the billing dimension child pages.
cc: @yetanothertw this should simplify your work a touch
Generative AI disclosure