-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
[Validator] Feature #36362 add Isin validator constraint #36368
[Validator] Feature #36362 add Isin validator constraint #36368
Conversation
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.
Interesting pull request! Just a few minor requests otherwise for me, this pull request is ready to be merged 🚀
Feature #36362 typo Fix PR feedbacks Fix coding standard ticket 36362 fix PR feedbacks Update src/Symfony/Component/Validator/Constraints/IsinValidator.php Co-Authored-By: Yannis Foucher <33806646+YaFou@users.noreply.github.com>
src/Symfony/Component/Validator/Resources/translations/validators.en.xlf
Outdated
Show resolved
Hide resolved
Hi @lmasforne |
Co-authored-by: Oskar Stark <oskarstark@googlemail.com>
@VincentLanglet seems to be ok ;) ; hope this PR will be merged soon |
@lmasforne Your PR have some CI-failures You need to fix https://fabbot.io/report/symfony/symfony/36368/46116925704192e42fd76f0f86fad8b60e028f65 Do you need help ? |
This PR was merged into the 5.2-dev branch. Discussion ---------- [Validator] Add Isin validator constraint Co-Authored-By: Yannis Foucher <33806646+YaFou@users.noreply.github.com> | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | yes | Deprecations? | no | Tickets | Fix #36362 | License | MIT | Doc PR | symfony/symfony-docs#13960 Rebase of #36368 I asked him by mail and he didn't have time to finish the PR and allowed me to do it. Commits ------- 8e1ffc8 Feature #36362 add Isin validator constraint
This PR was merged into the 5.2-dev branch. Discussion ---------- [Validator] Add Isin validator constraint Co-Authored-By: Yannis Foucher <33806646+YaFou@users.noreply.github.com> | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | yes | Deprecations? | no | Tickets | Fix #36362 | License | MIT | Doc PR | symfony/symfony-docs#13960 Rebase of symfony/symfony#36368 I asked him by mail and he didn't have time to finish the PR and allowed me to do it. Commits ------- 8e1ffc8b99 Feature #36362 add Isin validator constraint
merged as #37565 |
Add ISIN validator constraint
see here for more details https://en.wikipedia.org/wiki/International_Securities_Identification_Number