Skip to content

docs: document directed identity for non-interactive grants#777

Merged
brad-dow merged 2 commits intomainfrom
bd/docs-directed-identity
Apr 20, 2026
Merged

docs: document directed identity for non-interactive grants#777
brad-dow merged 2 commits intomainfrom
bd/docs-directed-identity

Conversation

@brad-dow
Copy link
Copy Markdown
Contributor

Fixes #DOC-34

Updated the following pages to document directed identity:

  • client-keys.mdx - key registry, client request body, grant requests flow, and sequence diagram intro
  • grants.mdx - clarify that directed identity clients provide their public key directly and don't require a key registry
  • auth.mdx - included directed identity as an option for incoming payment and quote grant requests

Also, bumps the open-payments-specifications submodule to pick up the latest spec changes, including directed identity

Checklist

  • PR title is prefixed with docs:
  • PR title or description includes a fixes #
  • You've run pnpm format and pnpm lint
  • You've reviewed Vale errors and made changes where appropriate

Updated client-keys.mdx, grants.mdx, and auth.mdx to reflect updates to OP specifications
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 16, 2026

Deploy Preview for openpayments ready!

Name Link
🔨 Latest commit 49e60c1
🔍 Latest deploy log https://app.netlify.com/projects/openpayments/deploys/69e688542a29d40008d19806
😎 Deploy Preview https://deploy-preview-777--openpayments.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.

Copy link
Copy Markdown
Contributor

@melissahenderson melissahenderson left a comment

Choose a reason for hiding this comment

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

Looks good overall. Do you think we should add directed identity to the glossary?


:::tip[Summary]
Client keys are unique identifiers that allow clients to sign Open Payments requests. They allow a client's identity to be verified through a public key registry.
Client keys are unique identifiers that allow clients to sign Open Payments requests and verify their identity.
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.

Who is "their"? Clients or requests?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch...I'll add "the client's identity".

Comment thread .gitignore Outdated
.DS_Store

# Claude Code
.claude
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.

Was this a Prettier addition?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I installed claude code locally for some experimentation, and I'm pretty sure it was an artifact from that. I pulled that back out, so it shouldn't show up anywhere else.

@brad-dow
Copy link
Copy Markdown
Contributor Author

Looks good overall. Do you think we should add directed identity to the glossary?

I waffled a bit on whether to include it or not. I feel like the rest of the glossary terms are foundational to Open Payments. Things like ASE and infrastructure pieces like auth server, etc. Directed identity feels like a layer deeper than those. However, because it's a "thing with a name" now across multiple pages it's worth including? I could go either way.

Enhanced some wording and fixed gitignore
@brad-dow brad-dow merged commit bef60a8 into main Apr 20, 2026
6 checks passed
@brad-dow brad-dow deleted the bd/docs-directed-identity branch April 20, 2026 22:21
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