-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[Form] extend the documentation on dynamic form modification #2238
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
Comments
Hi Dorthe! +1 for details on different event times (when to use what) We should review the document and make these changes, but only after #1842 has been merged in. Thanks! |
Ryan, that's perfectly fine that ordering is done in the template, but it should be noted somewhere. My English is sometimes ... interesting... but if I can support you somehow in preparing something, let me know :-) |
This issue is ready to be worked on - there are additional changes needed in #2464. |
I do believe the YAML configuration sample at the end of this page has some typos. |
@cheesemacfly what do you mean? |
@xabbuh Shouldn't the |
@cheesemacfly you're completely correct. There is already a fix for that in #2743 |
@wouterj you are right. Thanks for the information! |
I've just merged in that fix! I'm going to close this issue - my intention was to use #2464 going forward to follow the issues with this entry. Thanks! |
Hi guys, would you be kind, and share the link where I can find these additional documentation? Thank you in advance. |
Hi @barnibx, which additional documentation are you talking about? |
Hi @xabbuh, I realized after my comments, that I can check the modifications on each issue (#xxc), but I was wondering if these additions can be found in one place with some examples. Thanks for your question. |
Hey @barnibx - these have all been put into different parts in this article: http://symfony.com/doc/current/cookbook/form/dynamic_form_modification.html. We more or less addressed these issues, but we don't talk directly about, for example, when to use the different form events (that article uses FormEvents::PRE_SET_DATA several times). If there's something missing that you'd like to see, please definitely open an issue :). Cheers! |
HI @weaverryan, Thanks for your response. Thanks in advance. |
I suggest to add the following information either to the documentation or the cookbook
I am referring to http://symfony.com/doc/current/cookbook/form/dynamic_form_modification.html
The text was updated successfully, but these errors were encountered: