Skip to content

Conversation

nykopol
Copy link
Contributor

@nykopol nykopol commented Jan 6, 2014

Q A
Doc fix? yes
New docs? yes
Applies to 2.3+
Fixed tickets #3410 (task Form -> max_length)

@wouterj
Copy link
Member

wouterj commented Jan 6, 2014

👍

Copy link
Contributor

Choose a reason for hiding this comment

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

client-side -- .. so data must still be ...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks @bicpi It's fixed.

Copy link
Contributor

Choose a reason for hiding this comment

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

data, not datas :-)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed the last one !?

@weaverryan
Copy link
Member

Awesome work @nykopol. Keep it up - and thanks!

weaverryan added a commit that referenced this pull request Jan 9, 2014
…e (nykopol)

This PR was merged into the 2.3 branch.

Discussion
----------

[Reference][Form Types] Add "max_length" option in form type

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | yes
| Applies to    | 2.3+
| Fixed tickets | #3410 (task Form -> max_length)

Commits
-------

b53a227 fix last typo on max length option
d6800d3 fix typo in previous commit
391fe30 fix typo in previous commit
2156545 add max_length to form field type
@weaverryan weaverryan merged commit b53a227 into symfony:2.3 Jan 9, 2014
@nykopol nykopol deleted the missing_form_options branch January 9, 2014 04:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants