Skip to content

Remove unsupported Stripe Connect countries and add error handling#344

Merged
simonhamp merged 1 commit intomainfrom
fix-stripe-unsupported-countries
Apr 10, 2026
Merged

Remove unsupported Stripe Connect countries and add error handling#344
simonhamp merged 1 commit intomainfrom
fix-stripe-unsupported-countries

Conversation

@simonhamp
Copy link
Copy Markdown
Member

Summary

  • Removed 5 countries (IN, TW, KR, NG, NA) from the Stripe Connect supported countries list — these fail at the Stripe API level due to cross-border restrictions (US platform cannot create Express accounts) or missing card_payments capability
  • Added try/catch around StripeConnectService::createConnectAccount() in the onboarding controller to handle future Stripe InvalidRequestException errors gracefully with a user-friendly validation message instead of a 500
  • Cleaned up orphaned currency entries (INR, TWD, KRW, NGN, NAD) from CURRENCY_NAMES

Closes #45

Test plan

  • Unit tests verify removed countries are excluded from StripeConnectCountries
  • Feature tests verify IN and TW are rejected by validation
  • Feature test verifies Stripe InvalidRequestException is caught and returns a validation error
  • All 46 existing + new tests pass

🤖 Generated with Claude Code

…ndling

Remove IN, TW, KR, NG, and NA from the supported countries list as
Stripe does not support Express Connect accounts for these countries
from US-based platforms (cross-border restrictions or missing
card_payments capability). Also adds a try/catch around account
creation to handle any future Stripe rejections gracefully instead
of showing a 500 error.

Closes #45

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@simonhamp simonhamp marked this pull request as ready for review April 10, 2026 00:36
@simonhamp simonhamp merged commit 2ed0198 into main Apr 10, 2026
2 checks passed
@simonhamp simonhamp deleted the fix-stripe-unsupported-countries branch April 10, 2026 00:36
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