-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[Intl] Support ISO 3166-1 Alpha-3 country codes #32988
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
This a continuation of #32676 |
@derrabus Could you please complete your review? |
@terjebraten-certua I'm not familiar enough with the component to approve your changes, but I would trust that @ro0NL's review has been thorough. 😃 Status: Reviewed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with some minor CS comments
@fabpot I did all the changes you requested. |
Thank you @terjebraten-certua. |
…braten-certua) This PR was merged into the 4.4 branch. Discussion ---------- [Intl] Support ISO 3166-1 Alpha-3 country codes | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #20313 | Doc PR | symfony/symfony-docs#12105 | License | MIT This is a pre-requiste for fixing #20313 and only handles the changes to the INTL component. Commits ------- 848f60e Support ISO 3166-1 Alpha-3 country codes
…axperrimond) This PR was squashed before being merged into the 5.1-dev branch (closes #35116). Discussion ---------- [Validator] Add alpha3 option to country constraint | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | yes | Deprecations? | no | Tickets | | License | MIT | Doc PR | symfony/symfony-docs#12857 A following of #33791 and #32988 to add `alpha3` option also to `Country` constraint in the validator component. Commits ------- d6f34a5 [Validator] Add alpha3 option to country constraint
This is a pre-requiste for fixing #20313 and only handles the changes to the INTL component.