Skip to content

docs(stripe): clarify api_key_id vs api_key_name vault options#595

Merged
burmecia merged 6 commits intomainfrom
copilot/fix-query-vault-failure
Apr 10, 2026
Merged

docs(stripe): clarify api_key_id vs api_key_name vault options#595
burmecia merged 6 commits intomainfrom
copilot/fix-query-vault-failure

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 10, 2026

  • Revert "With Vault (using key ID)" + "With Vault (using key name)" back to original single "With Vault" tab
  • Add descriptive bullet list inside the "With Vault" tab clarifying api_key_id vs api_key_name
  • Add comment to the 3rd parameter of vault.create_secret

Copilot AI linked an issue Apr 10, 2026 that may be closed by this pull request
2 tasks
Copilot AI changed the title [WIP] Fix query vault failed error during stripe integration docs(stripe): clarify api_key_id vs api_key_name vault options Apr 10, 2026
Copilot AI requested a review from burmecia April 10, 2026 01:27
…ith Vault tab

Agent-Logs-Url: https://github.com/supabase/wrappers/sessions/84fc15a3-8220-4d92-83a6-641528a1b1b3

Co-authored-by: burmecia <19306324+burmecia@users.noreply.github.com>
@burmecia burmecia marked this pull request as ready for review April 10, 2026 01:43
Copilot AI review requested due to automatic review settings April 10, 2026 01:43
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Stripe wrapper documentation to clarify how to use Vault-backed credentials, specifically distinguishing the api_key_id and api_key_name server options.

Changes:

  • Consolidates Vault connection guidance into a single “With Vault” tab.
  • Adds an explanation of api_key_id vs api_key_name (including precedence).
  • Improves the Vault secret-creation snippet with parameter comments and adds a query to retrieve the secret UUID later.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
… vs api_key_name

Agent-Logs-Url: https://github.com/supabase/wrappers/sessions/bd13c673-32e0-4c55-9ffd-ff4e43c3832a

Co-authored-by: burmecia <19306324+burmecia@users.noreply.github.com>
…result to key_id

Agent-Logs-Url: https://github.com/supabase/wrappers/sessions/84b96a3b-ebb1-46a0-9b6f-2255ba7bf03a

Co-authored-by: burmecia <19306324+burmecia@users.noreply.github.com>
@burmecia burmecia requested a review from imor April 10, 2026 02:09
@burmecia burmecia merged commit 926df23 into main Apr 10, 2026
5 checks passed
@burmecia burmecia deleted the copilot/fix-query-vault-failure branch April 10, 2026 02:33
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.

query vault failed

4 participants