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

4 participants