Skip to content

chore: remove deprecated checkoutcom-flow and stripe-payment-element from CreateOrderInputDTO#1988

Closed
fedemint wants to merge 2 commits into
mainfrom
devin/replace-deprecated-payment-methods-with-card-sdk
Closed

chore: remove deprecated checkoutcom-flow and stripe-payment-element from CreateOrderInputDTO#1988
fedemint wants to merge 2 commits into
mainfrom
devin/replace-deprecated-payment-methods-with-card-sdk

Conversation

@fedemint

Copy link
Copy Markdown

Description

Removes the deprecated checkoutcom-flow and stripe-payment-element payment.method enum options from CreateOrderInputDTO in packages/wallets/src/openapi.json. The existing card option covers the same card-payment flows, and this aligns the public SDK spec with the docs deprecation messaging.

Note: packages/client/base/src/lib/hosted-checkout/Order.ts still lists these values because it mirrors the order response schema, and the API currently still emits processor-specific method values in order objects.

Test plan

  • Verified the file is valid JSON after the change
  • Confirmed card remains a valid payment.method option in both anyOf branches of CreateOrderInputDTO
  • Confirmed checkoutcom-flow and stripe-payment-element are no longer present in CreateOrderInputDTO.payment oneOf entries

Package updates

  • This is a spec-only change in packages/wallets/src/openapi.json and does not change runtime code or public package APIs, so no changeset is required.

Link to Devin session: https://crossmint.devinenterprise.com/sessions/0e2ad01383a94c8282aa5dc3493084ca
Requested by: @fedemint

… CreateOrderInputDTO in openapi spec

The existing  payment method option covers the same card-payment flows. This keeps the public SDK OpenAPI spec aligned with the docs deprecation messaging.

Co-Authored-By: Fede Sarquis <fede@paella.dev>
@fedemint fedemint self-assigned this Jul 23, 2026
@devin-ai-integration

Copy link
Copy Markdown
Contributor
Original prompt from Fede

SYSTEM:
=== BEGIN THREAD HISTORY (in #prog-documentation) ===
<most_recent_message>
Fede (U08TJFT76H5): @Devin check this thread:
<https://crossmint.slack.com/archives/C05KVC0AUL8/p1784805601731299?thread_ts=1784805601.731299&amp;amp;cid=C05KVC0AUL8|https://crossmint.slack.com/archives/C05KVC0AUL8/p1784805601731299?thread_ts=1784805601.731299&amp;amp;cid=C05KVC0AUL8>

Coud you review the docs and public resources to try and find any reference that would lead an external party to use the code in this thread incorrectly? Are there any mentions of it in our docs or code snippets with it?
</most_recent_message>
=== END THREAD HISTORY ===

Thread URL: https://crossmint.slack.com/archives/C04CU001TGA/p1784817707085829?thread_ts=1784817707.085829&amp;cid=C04CU001TGA

The latest message is the one right above that tagged you. The <most_recent_message> is the message that you should use to guide your goals + task for this session, and you should use the rest of the slack thread as context.

@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@changeset-bot

changeset-bot Bot commented Jul 23, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: fff1426

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 9 packages
Name Type
@crossmint/wallets-sdk Patch
@crossmint/wallets-quickstart-devkit Patch
@crossmint/wallets-playground-react Patch
@crossmint/client-sdk-react-base Patch
@crossmint/client-sdk-react-native-ui Patch
@crossmint/client-sdk-react-ui Patch
@crossmint/wallets-playground-expo Patch
@crossmint/auth-ssr-nextjs-demo Patch
@crossmint/client-sdk-nextjs-starter Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Co-Authored-By: Fede Sarquis <fede@paella.dev>
@fedemint fedemint closed this Jul 23, 2026
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