Skip to content

Distinguish OAuth providers (outbound) from OAuth clients (inbound)#33

Merged
micheleRP merged 1 commit into
mainfrom
docs/clarify-oauth-direction
May 27, 2026
Merged

Distinguish OAuth providers (outbound) from OAuth clients (inbound)#33
micheleRP merged 1 commit into
mainfrom
docs/clarify-oauth-direction

Conversation

@micheleRP
Copy link
Copy Markdown
Contributor

@micheleRP micheleRP commented May 27, 2026

Summary

  • Lead the OAuth Provider (mcp/pages/oauth-providers.adoc) and OAuth Client (integrations/pages/remote-mcp-clients.adoc) pages with directional IMPORTANT blocks so readers grasp on first scan that providers govern outbound authentication (ADP → upstream services) and clients govern inbound authentication (external apps → ADP).
  • Remove the now-redundant mid-page NOTE on remote-mcp-clients.adoc; sweep authauthentication in prose across both pages plus user-delegated-oauth.adoc (CLI flag names like --auth-endpoint preserved); replace internal proto jargon with API definitions in the provider callout.
  • Promote Remote MCP Clients to the top of the Integrations nav and shorten its label — it is the canonical onboarding for remote-MCP integrations, and the other entries under Integrations are still stubs.
  • Companion glossterm rewrites (OAuth provider / OAuth client hover text) ride along on Add ADP-specific glossary terms for GA docs#1691; tooltips render the new inbound/outbound framing once that PR merges.

Preview pages

Test plan

  • npm run build completes cleanly (verified locally — no new warnings on the four touched pages).
  • Visit the three preview pages above and confirm:
    • oauth-providers/ — new IMPORTANT block at the top of the page, bullets call out outbound/inbound, "API definitions" replaces "proto".
    • remote-mcp-clients/ — new IMPORTANT block before "When to use this"; no duplicate NOTE mid-page; prose now reads "authentication" throughout.
    • user-delegated-oauth/ — "In the authentication section" and "needs this server's authentication" both read correctly.
  • Left sidebar: under Integrations, Remote MCP Clients is the first entry and its label is just "Remote MCP Clients" (no parenthetical).
  • Once Add ADP-specific glossary terms for GA docs#1691 merges, glossterm:OAuth provider[] / glossterm:OAuth client[] tooltips on the glossary page render the new inbound/outbound hover text.

🤖 Generated with Claude Code

Lead the OAuth Provider and OAuth Client pages with directional
authentication framing so readers grasp the distinction on first scan.
Add a reciprocal IMPORTANT block to remote-mcp-clients.adoc, remove the
redundant mid-page NOTE, sweep "auth" -> "authentication" in prose
across both pages plus user-delegated-oauth.adoc, and replace internal
"proto" jargon with "API definitions" in the provider callout.

Promote Remote MCP Clients to the top of the Integrations nav and
shorten its label - it is the canonical onboarding for remote-MCP
integrations and the other entries are still stubs.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@micheleRP micheleRP requested a review from a team as a code owner May 27, 2026 04:15
@netlify
Copy link
Copy Markdown

netlify Bot commented May 27, 2026

Deploy Preview for redpanda-agentic-data-plane ready!

Name Link
🔨 Latest commit 4484c79
🔍 Latest deploy log https://app.netlify.com/projects/redpanda-agentic-data-plane/deploys/6a166fd1c841df00084ab532
😎 Deploy Preview https://deploy-preview-33--redpanda-agentic-data-plane.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@micheleRP micheleRP merged commit c1b7d88 into main May 27, 2026
4 checks passed
@micheleRP micheleRP deleted the docs/clarify-oauth-direction branch May 27, 2026 04:22
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