Skip to content

Update CPS placeholder pricing - #7552

Open
shainaraskas wants to merge 4 commits into
mainfrom
cps-billing-future
Open

Update CPS placeholder pricing#7552
shainaraskas wants to merge 4 commits into
mainfrom
cps-billing-future

Conversation

@shainaraskas

Copy link
Copy Markdown
Member

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

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

@shainaraskas
shainaraskas requested a review from tiamliu July 23, 2026 19:29
@shainaraskas
shainaraskas requested a review from a team as a code owner July 23, 2026 19:29
@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Elastic Docs AI PR menu

Check 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.

@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

@github-actions

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.

@eedugon eedugon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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 yetanothertw 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.

Thanks for the ping! I'll make sure to pull these changes into my branch.

Left a tiny comment for simplification. 🌻

Comment thread deploy-manage/_snippets/cps-billing.md Outdated
Co-authored-by: Vlada Chirmicci <vlada.chirmicci@elastic.co>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Docs review summary

Focus areas

  • Style and clarity: Vale flagged only a pre-existing execute word-choice suggestion (unrelated to this diff, line untouched) and a routine mapped_pages warning (metadata unchanged by this PR). One inline suggestion posted on deploy-manage/_snippets/cps-billing.md for 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: preview blocks that gated the now-removed per-product CPS sections were deleted along with those sections — no orphaned applies_to blocks 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#1500 by 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.md is transcluded from two other pages (cross-project-search-config.md and serverless-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

Comment thread deploy-manage/_snippets/cps-billing.md Outdated
Comment thread deploy-manage/_snippets/cps-billing.md Outdated
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