Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[Reference][Form] Describe the syntax of format option using a link of DateTimeType #9788

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
May 17, 2018
Merged

Conversation

elementaire
Copy link
Contributor

@elementaire elementaire commented May 16, 2018

I've added a link to describe how to write a valid format. It is the same link from date and birthday types.

@elementaire elementaire changed the title Update datetime.rst [Reference][Form] Describe the syntax of format option using a link of DateTimeType May 17, 2018
@javiereguiluz
Copy link
Member

Let's ping @vudaltsov as a Symfony Form expert to help us here. The PHPdoc of the DateTimeType class has a lot of information. Also, the ticket opened at ICU project was answered six years ago: http://icu-project.org/trac/ticket/9421 Can we review the PHPdoc of DateTimeType and see if it's still relevant or outdated? Thanks!

@elementaire
Copy link
Contributor Author

The same link is used in DateType and BirthdayType docs. It comes from this template of this documentation. Depends on the answer of @vudaltsov, maybe it should be updated too.

@vudaltsov
Copy link
Contributor

@javiereguiluz , I think we should create a separate issue to see if the PHPdoc is still relevant.
What @elementaire proposed can be done in any case.

@javiereguiluz javiereguiluz added this to the 2.7 milestone May 17, 2018
@javiereguiluz
Copy link
Member

Thanks for this improvement and congrats on your first Symfony Docs contribution! Thanks to reviewers too!

@javiereguiluz javiereguiluz merged commit 8b361ef into symfony:2.7 May 17, 2018
javiereguiluz added a commit that referenced this pull request May 17, 2018
…ing a link of DateTimeType (elementaire)

This PR was merged into the 2.7 branch.

Discussion
----------

[Reference][Form] Describe the syntax of format option using a link of DateTimeType

I've added a link to describe how to write a valid format. It is the same link from date and birthday types.

Commits
-------

8b361ef Update datetime.rst
@elementaire
Copy link
Contributor Author

Should i do something for this to be merged in recent versions of docs too?

@javiereguiluz
Copy link
Member

@elementaire there's no need to create new PRs. From time to time we merge all changes into the upper (maintained) branches. I've just do that, so your change is now in 2.7, 2.8, 3.4, 4.0, 4.1 and master. In a few hours, when symfony.com docs get rebuilt and caches expire, your change will be published. Cheers!

@elementaire
Copy link
Contributor Author

Great :)

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
Morty Proxy This is a proxified and sanitized view of the page, visit original site.