Skip to content

fix: temporarily disable stripe_v2 configuration from new admin#238

Merged
brettflorio merged 4 commits intomainfrom
copilot/disable-stripe-v2-configuration
Mar 30, 2026
Merged

fix: temporarily disable stripe_v2 configuration from new admin#238
brettflorio merged 4 commits intomainfrom
copilot/disable-stripe-v2-configuration

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 30, 2026

  • Add stripe_v2 to the oauthGateways array in PaymentsApiPaymentMethodForm.ts to show the same legacy-admin redirect message as stripe_connect
  • Update the test in PaymentsApiPaymentMethodForm.test.ts to include stripe_v2 in the list of gateways that should show the warning
  • Fix commit messages to follow conventional commit format (fix(payments-api-payment-method-form): ...)

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 30, 2026

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copilot AI changed the title [WIP] Temporarily disable stripe_v2 configuration from new admin Temporarily disable stripe_v2 configuration from new admin Mar 30, 2026
Copilot AI requested a review from pheekus March 30, 2026 13:56
@pheekus pheekus marked this pull request as ready for review March 30, 2026 14:05
@pheekus
Copy link
Copy Markdown
Contributor

pheekus commented Mar 30, 2026

@copilot this project requires the use of conventional commit format. Can you fix the commit messages? The prefix needs to be "fix" and the scope is the custom element name in kebab-case.

Copilot AI and others added 2 commits March 30, 2026 14:11
This reverts commit daf7354.

Co-authored-by: pheekus <47783757+pheekus@users.noreply.github.com>
…configuration from new admin

Agent-Logs-Url: https://github.com/Foxy/foxy-elements/sessions/6a771a76-1acb-4bcb-bdad-1a68641f3547

Co-authored-by: pheekus <47783757+pheekus@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 30, 2026

@copilot this project requires the use of conventional commit format. Can you fix the commit messages? The prefix needs to be "fix" and the scope is the custom element name in kebab-case.

Done in 17f8d3c — the commit now uses fix(payments-api-payment-method-form): temporarily disable stripe_v2 configuration from new admin. The earlier commits with the incorrect messages remain in history since force-push wasn't available, but the tip commit has the correct conventional format.

Copilot AI requested a review from pheekus March 30, 2026 14:14
@pheekus pheekus changed the title Temporarily disable stripe_v2 configuration from new admin fix: temporarily disable stripe_v2 configuration from new admin Mar 30, 2026
@pheekus pheekus requested a review from brettflorio March 30, 2026 14:42
@brettflorio brettflorio merged commit b820ba9 into main Mar 30, 2026
6 of 7 checks passed
@brettflorio brettflorio deleted the copilot/disable-stripe-v2-configuration branch March 30, 2026 19:27
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.

Temporarily disable stripe_v2 configuration from new admin

4 participants