Skip to content

Alter UCP Commands to match param specs - #297

Merged
ericgan-stripe merged 5 commits into
stripe:kreese/ucp-checkout-commandsfrom
ericgan-stripe:ericgan/ucp-business-param
Sep 9, 2026
Merged

ericgan-stripe merged 5 commits into
stripe:kreese/ucp-checkout-commandsfrom
ericgan-stripe:ericgan/ucp-business-param

Conversation

@ericgan-stripe

@ericgan-stripe ericgan-stripe commented Sep 4, 2026 •

Copy link
Copy Markdown
Contributor

link-cli ucp catalog search

  • Changed from --network-id to --business
  • Changed from --sku to --id

link-cli ucp checkout create

  • Changed from --network-id to --business
  • Changed from --sku_id to --id

link-cli ucp checkout complete

  • Changed --shared-payment-token to --spend-request-id
  • Added --business option

@ericgan-stripe
ericgan-stripe requested a review from a team as a code owner September 4, 2026 21:05
@ericgan-stripe ericgan-stripe changed the title Alter catalog and checkout create commands Alter UCP Commands to match param specs Sep 7, 2026
ericgan-stripe and others added 2 commits September 7, 2026 17:08
Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>
Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

@kreese-stripe kreese-stripe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm, one non-blocking comment

const availability =
product.availability ?? firstVariant?.availability?.status;
const networkId = product.profile_id ?? firstVariant?.profile_id;
const business = product.profile_id ?? firstVariant?.profile_id;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

not blocking: since there aren't any prod users of this API/these commands today, wonder if we should also update the API response schema? So we don't have to do this translation on the "frontend". Theoretically users could eventually use not just the CLI but the SDK, which doesn't appear to have this translation

@ericgan-stripe
ericgan-stripe merged commit 8e0dc71 into stripe:kreese/ucp-checkout-commands Sep 9, 2026
2 checks passed
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.

2 participants