Skip to content

Update the states constant for the states to be available per country#73

Open
Kevin14Swoo wants to merge 5 commits intomainfrom
feature/country-states
Open

Update the states constant for the states to be available per country#73
Kevin14Swoo wants to merge 5 commits intomainfrom
feature/country-states

Conversation

@Kevin14Swoo
Copy link

@Kevin14Swoo Kevin14Swoo commented Feb 25, 2025


Copy link
Member

@chinleung chinleung left a comment

Choose a reason for hiding this comment

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

@Kevin14Swoo Pourquoi on fait cette modification à la structure ? Le problème que je vois long term avec cette implémentation c'est qu'on ne vas pas pouvoir faire State::QUEBEC admettons. Il va falloir faire CanadaState::QUEBEC.

@Kevin14Swoo
Copy link
Author

@Kevin14Swoo Pourquoi on fait cette modification à la structure ? Le problème que je vois long term avec cette implémentation c'est qu'on ne vas pas pouvoir faire State::QUEBEC admettons. Il va falloir faire CanadaState::QUEBEC.

Avant y avait juste les states de Australia et Québec dans le enum States. Quand j'ai ajouté les states complete de Canada, on a un doublon de NT
Screenshot 2025-03-03 at 5 21 58 AM

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