Source-verify ADP docs: add BambooHR/NetSuite, refresh governance and BYOC#16
Merged
Conversation
… BYOC Verified every claim against cloudv2 origin/main (commit 992c33f099): - New managed MCP deep-dives: BambooHR (5 tools, subdomain config), NetSuite (7 read-only tools, accountId + embedded user OAuth) - Managed catalog: 36 registry-backed types, added Azure AD and Morningstar Portfolio/Securities entries, moved Metabase to Database - Governance overview: rewrote to match shipping UI (3 KPI tabs, real filter set including cost-type and token-type, Top users panel, hourly/daily bucketing rule) - BYOC prerequisites: AWS-only enforcement (verified at controlplane-api defaulter.go), Crossplane-managed IAM patterns, ADP component list, added an About Crossplane explainer - BYOC quickstart: scoped to verified content; defers click-by-click UI steps to a later UI-walkthrough pass - create-server.adoc: fixed `rpk ai mcp create` examples to use `--name <name>` (the source CLI requires the flag; positional name fails). Same fix applied across ironclad, metabase, ramp, workday, zendesk deep-dive pages - nav.adoc: added BambooHR and NetSuite catalog entries Style pass: list parallelism (closing periods on all items or none), removed internal-jargon leakage (`cloudv2`, `ManagedMCPCategory`), removed writer-facing meta-prose, normalized "Where to go next" / "Investigate further" headings to "Next steps". Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
✅ Deploy Preview for redpanda-agentic-data-plane ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
…rgon Review feedback from PR #16: - Date-range list, KPI tabs table, Filter table, Agents column headers, cost/token type values: removed bold from field names and option values per the new bolding rule (bold = click targets only) - KPI tab labels: dropped all-caps + bold (heavy, weird-looking) for plain sentence case - Em-dashes replaced with colons / parentheses / sentence breaks per the team style guide. Includes register-remote.adoc (touched here per user request even though it was outside the original PR scope). - managed-catalog.adoc: replaced "—" placeholder cells in the Deep dive column with empty cells (no character) - create-server.adoc and register-remote.adoc: renamed "Out of scope" → "Related topics" with a clearer intro. The same pattern exists on 17 other ADP pages and could be swept in a follow-up. New memory: feedback_bolding_rule.md. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Renames the remaining 16 "Out of scope" sections to clearer headings based on what each section actually contains: - "Limitations" (14 pages) where items describe feature or page capability limits — what the page or feature deliberately doesn't do. - "Related topics" (test-tools.adoc, user-delegated-oauth.adoc) where items are pointers to other docs, with brief context. Also fixed em-dashes to colons in the rewritten sections (kafka, jira, sql, openapi) per the team style guide. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Verifies and refines the ADP docs handed off mid-task from a Codex session. Every claim was checked against
cloudv2 origin/main(commit 992c33f099).accountId+ embedded user OAuth).apps/controlplane-api/defaults/defaulter.go(Provider == AWSis required); Crossplane-managed IAM patterns (aigw-<id>*,adp-<id>*); ADP component list. Added an About Crossplane explainer because the IAM patterns are easier to reason about with the Crossplane context.rpk ai mcp createexamples (acrosscreate-server.adoc,ironclad,metabase,ramp,workday,zendesk): fixed to use--name <name>. The cobra command marks--nameas required, so positional-name examples were broken.cloudv2,ManagedMCPCategory), removed writer-facing meta-prose, normalized Where to go next / Investigate further headings to Next steps.npm run buildis clean for every file in this PR.Preview pages
Trivial
rpk --nameflag fix only onironclad,metabase,ramp,workday,zendeskdeep-dive pages — diff is a single line each, no preview link needed.Test plan
rpk ai mcp create --name <name>is the syntax in every CLI example across MCP managed pages andcreate-server.adoc🤖 Generated with Claude Code