Skip to content

Add authentication command for stripe that accepts a JWT#7935

Open
dengjeffrey wants to merge 1 commit into
mainfrom
jd/shop-stripe-auth
Open

Add authentication command for stripe that accepts a JWT#7935
dengjeffrey wants to merge 1 commit into
mainfrom
jd/shop-stripe-auth

Conversation

@dengjeffrey

@dengjeffrey dengjeffrey commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

WHY are these changes introduced?

Introduces a new command as discussed. This command will allow an agent to pass their provided signup JWT to auth into a store they are signing up for.

Works in tandem with https://github.com/shop/world/pull/845960

WHAT is this pull request doing?

  1. Pass along a parameter to the Oauth GET request
  2. Add process.env.SHOPIFY_STORE_AUTH_APP_CLIENT_ID as an override for STORE_AUTH_APP_CLIENT_ID because local development uses a different ID

How to test your changes?

https://docs.google.com/document/d/17Jxc47OA58hH7gLBj9Cu-1x-bw0PGi4kLiAp0hsIsho/edit?tab=t.0#bookmark=id.jodpy9qjzb9j

Post-release steps

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes
  • I've considered analytics changes to measure impact
  • The change is user-facing — I've identified the correct bump type (patch for bug fixes · minor for new features · major for breaking changes) and added a changeset with pnpm changeset add

@dengjeffrey dengjeffrey requested review from a team as code owners June 26, 2026 03:20
@dengjeffrey dengjeffrey self-assigned this Jun 26, 2026
@dengjeffrey dengjeffrey force-pushed the jd/shop-stripe-auth branch from 1090ba8 to 472c202 Compare June 26, 2026 04:59
@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label Jun 26, 2026
@dengjeffrey dengjeffrey force-pushed the jd/shop-stripe-auth branch from 472c202 to 4e2b9c1 Compare June 26, 2026 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant