Skip to content

[Validator] add new Timezone validation constraint #30900

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

Merged
merged 1 commit into from
Apr 6, 2019

Conversation

hhamon
Copy link
Contributor

@hhamon hhamon commented Apr 6, 2019

Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
License MIT
Doc PR symfony/symfony-docs#11317

Rework of #22262.

@hhamon hhamon mentioned this pull request Apr 6, 2019
2 tasks
@hhamon hhamon force-pushed the tz_validator branch 3 times, most recently from 8e03af4 to 6b6c8d0 Compare April 6, 2019 14:15
@nicolas-grekas nicolas-grekas added this to the next milestone Apr 6, 2019
@hhamon
Copy link
Contributor Author

hhamon commented Apr 6, 2019

Also documentation PR is on its way as well ;)

@hhamon hhamon changed the title Add new TimezoneValidator [Validator] add new Timezone validation constraint Apr 6, 2019
Copy link
Member

@xabbuh xabbuh left a comment

Choose a reason for hiding this comment

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

with a minor change request

@hhamon hhamon force-pushed the tz_validator branch 2 times, most recently from f13c04b to e619668 Compare April 6, 2019 16:25
@fabpot
Copy link
Member

fabpot commented Apr 6, 2019

Thank you @hhamon.

@fabpot fabpot merged commit 536e53f into symfony:master Apr 6, 2019
fabpot added a commit that referenced this pull request Apr 6, 2019
…phansys)

This PR was merged into the 4.3-dev branch.

Discussion
----------

[Validator] add new `Timezone` validation constraint

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| License       | MIT
| Doc PR        | symfony/symfony-docs#11317

Rework of #22262.

Commits
-------

536e53f [Validator] add new `Timezone` validation constraint.
@hhamon hhamon deleted the tz_validator branch April 6, 2019 18:53
wouterj added a commit to symfony/symfony-docs that referenced this pull request Apr 7, 2019
…onstraint. (hhamon, javiereguiluz)

This PR was merged into the master branch.

Discussion
----------

[Validator] add documentation for the new `Timezone` constraint.

This is the documentation related to PR symfony/symfony#30900

Commits
-------

8e33d47 Fixes classification
5ec9159 Fix option
ec7bd40 Revert country_code to countryCode
9cbeb75 Fixes options list table
c5681bf Add more explanations
b37bc78 Minor fixes
d564218 Minor fixes
ac6282f Listed the geographical zones defined by PHP
135c5ff Minor fixes
b0b1498 [Validator] add documentation for the new `Timezone` constraint.
@ro0NL
Copy link
Contributor

ro0NL commented Apr 11, 2019

do we still consider #22262 (review)?

@nicolas-grekas nicolas-grekas modified the milestones: next, 4.3 Apr 30, 2019
@fabpot fabpot mentioned this pull request May 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants