Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Validator] support DateTimeInterface instances for times #21106

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

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Dec 30, 2016

Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets
License MIT
Doc PR

The same has already been done for the Date and DateTime constraints in #18759.

@nicolas-grekas
Copy link
Member

👍

@fabpot
Copy link
Member

fabpot commented Jan 2, 2017

Thank you @xabbuh.

@fabpot fabpot merged commit 1fa9276 into symfony:master Jan 2, 2017
fabpot added a commit that referenced this pull request Jan 2, 2017
…mes (xabbuh)

This PR was merged into the 3.3-dev branch.

Discussion
----------

[Validator] support DateTimeInterface instances for times

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets |
| License       | MIT
| Doc PR        |

The same has already been done for the `Date` and `DateTime` constraints in #18759.

Commits
-------

1fa9276 support DateTimeInterface instances for times
@xabbuh xabbuh deleted the time-constraint-datetimeinterface-support branch January 2, 2017 20:13
@nicolas-grekas nicolas-grekas modified the milestone: 3.x Mar 24, 2017
@fabpot fabpot mentioned this pull request May 1, 2017
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.

4 participants