Skip to content

Add BR state dropdown to address form#12848

Merged
toluo-stripe merged 3 commits into
masterfrom
tolu/br-state-only
Apr 6, 2026
Merged

Add BR state dropdown to address form#12848
toluo-stripe merged 3 commits into
masterfrom
tolu/br-state-only

Conversation

@toluo-stripe

@toluo-stripe toluo-stripe commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

Summary

Add state dropdown for Brazil using data from https://stripe.sourcegraphcloud.com/r/stripe-internal/mint/-/blob/pay-server/stripe-js-v3/src/lib/hostedPaymentUi/components/AddressInput/addressData.ts

Built off of this PR

Motivation

https://jira.corp.stripe.com/browse/RUN_MOBILESDK-5287

Testing

  • Added tests
  • Modified tests
  • Manually verified

Screenshots

Screen.Recording.2026-04-06.at.12.24.51.PM.mov

Changelog

  • [Added] Added state dropdown support for BR.

toluo-stripe and others added 3 commits April 6, 2026 12:10
Convert Brazil's state field from free-text input to a dropdown with 27
predefined state codes, preventing users from entering full state names
that fail server-side validation.

Fixes RUN_MOBILESDK-5287

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Committed-By-Agent: claude
@toluo-stripe toluo-stripe marked this pull request as ready for review April 6, 2026 17:47
@toluo-stripe toluo-stripe requested review from a team as code owners April 6, 2026 17:47
Comment thread CHANGELOG.md

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Feels like there would be a merge conflict, and might worth a rebase.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

My bad, I didn't notice this

@toluo-stripe toluo-stripe merged commit dd922ca into master Apr 6, 2026
22 checks passed
@toluo-stripe toluo-stripe deleted the tolu/br-state-only branch April 6, 2026 17:55
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