Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
[FrameworkBundle] Replace render() with new renderForm() in form documentation#15105
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Nyholm commentedMar 15, 2021
Thank you. Im not too confident/knowledgeable about this feature, but should it be applied everywhere? Or is it only for some specific use cases? |
MrYamous commentedMar 15, 2021
I think both are possible. |
xabbuh commentedMar 26, 2021
IMO we should document this only once. Not sure though what would be the best place for it. |
javiereguiluz commentedApr 9, 2021
@MrYamous thanks a lot for taking care of updating this! Some other changes were required to finish this update. Describing them in this PR was complex and I tried, but failed, to add some commits to this Pull Request. That's why I've created#15217 based on this PR. The new PR contains your original commit, so you get full credit for your work. That's why I'm closing this one. Thanks! |
…Form() in form documentation (MrYamous, javiereguiluz)This PR was submitted for the 5.4 branch but it was merged into the 5.3 branch instead.Discussion----------[Form][FrameworkBundle] Replace render() with new renderForm() in form documentationThis adds some missing changes to#15105 andfixes#14844.Commits-------fdb7b67 Update code to the latest helper behavior773e816 More changes for forms4b259cc Replace render by new renderForm method in form documentation
This is the documentation for#14844