Skip to content
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] Added docs about the format option #6316

Merged
merged 1 commit into from
Mar 5, 2016
Merged

[Validator] Added docs about the format option #6316

merged 1 commit into from
Mar 5, 2016

Conversation

dosten
Copy link
Contributor

@dosten dosten commented Mar 1, 2016

Q A
Doc fix? no
New docs? yes, see symfony/symfony#17553
Applies to 3.1+
Fixed tickets #6313

@dosten
Copy link
Contributor Author

dosten commented Mar 1, 2016

ping @xabbuh

.. versionadded:: 3.1
The ``format`` option was introduced in Symfony 3.1.

This option allows to validate a custom date format. See `date_create_from_format`_ for formatting options.
Copy link
Member

Choose a reason for hiding this comment

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

I would change the label to DateTime:createFromFormat():

See `DateTime::createFromFormat <date_create_from_format>`_ for [...]

@dosten
Copy link
Contributor Author

dosten commented Mar 1, 2016

@xabbuh comments addressed

.. versionadded:: 3.1
The ``format`` option was introduced in Symfony 3.1.

This option allows to validate a custom date format. See `DateTime::createFromFormat <date_create_from_format>`_ for formatting options.
Copy link
Member

Choose a reason for hiding this comment

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

Can you please the line after the first word that crosses the 72nd character?

@xabbuh
Copy link
Member

xabbuh commented Mar 1, 2016

👍 I just left some very minor comment.

@dosten
Copy link
Contributor Author

dosten commented Mar 1, 2016

@xabbuh done

@xabbuh
Copy link
Member

xabbuh commented Mar 1, 2016

Great! :)

@wouterj wouterj merged commit f8d1e82 into symfony:master Mar 5, 2016
wouterj added a commit that referenced this pull request Mar 5, 2016
This PR was merged into the master branch.

Discussion
----------

[Validator] Added docs about the format option

| Q             | A
| ------------- | ---
| Doc fix?      | no
| New docs?     | yes, see symfony/symfony#17553
| Applies to    | 3.1+
| Fixed tickets | #6313

Commits
-------

f8d1e82 Added docs about the format option
wouterj added a commit that referenced this pull request Mar 5, 2016
@wouterj
Copy link
Member

wouterj commented Mar 5, 2016

Thanks for the docs @dosten!

Fyi, I've changed the external link to use the custom :phpmethod: role in 6e73637.

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.

None yet

3 participants