Skip to content

Missing template at documentation #3705

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

Closed
crashboybr opened this issue Mar 21, 2014 · 4 comments
Closed

Missing template at documentation #3705

crashboybr opened this issue Mar 21, 2014 · 4 comments

Comments

@crashboybr
Copy link

At http://symfony.com/doc/current/cookbook/form/dynamic_form_modification.html

in the 3rd example (Dynamic generation for submitted Forms), where they talk about the AJAX request (Twig example) is missing this template:
https://github.com/symfony/symfony-docs/blob/master/cookbook/form/dynamic_form_modification_ajax_js.rst.inc

The associated template uses some JavaScript to update the position form field according to the current selection in the sport field:
Twig
{# src/Acme/DemoBundle/Resources/views/Meetup/create.html.twig #}
{{ form_start(form) }}
{{ form_row(form.sport) }} {# <select id="meetup_sport" ... #}
{{ form_row(form.position) }} {# <select id="meetup_position" ... #}
{# ... #}
{{ form_end(form) }}

.. include:: /cookbook/form/dynamic_form_modification_ajax_js.rst.inc

@wouterj
Copy link
Member

wouterj commented Mar 21, 2014

Thank you for reporting the problem!

However, this was already fixed by someone ( #3698 ). It seems like the site isn't rebuilded. @javiereguiluz

@xabbuh
Copy link
Member

xabbuh commented Mar 22, 2014

@wouterj It was fixed in 2.3 (see http://symfony.com/doc/2.3/cookbook/form/dynamic_form_modification.html) and hasn't been merged up the branches yet.

@wouterj
Copy link
Member

wouterj commented Mar 22, 2014

Thank you @xabbuh, they are merged now.

@crashboybr it should be fixed within an hour!

@wouterj wouterj closed this as completed Mar 22, 2014
@xabbuh
Copy link
Member

xabbuh commented Mar 22, 2014

@wouterj Thanks a lot! Now, testing the 2.4 and master branches locally is now much easier since the now included Makefile file doesn't cause conflicts when switching branches! 👍

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

No branches or pull requests

3 participants