Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

[2.1] Fixed build errors #2445

Merged
merged 3 commits into from

2 participants

@WouterJ
Collaborator
Q A
Doc fix? yes
New docs? no
Applies to 2.1+

There were some build errors. This PR, among with other PRs on some 3th party bundles, tries to fix them all.

@WouterJ WouterJ commented on the diff
cookbook/form/dynamic_form_modification.rst
@@ -161,12 +161,11 @@ How to Dynamically Generate Forms based on user data
====================================================
Sometimes you want a form to be generated dynamically based not only on data
-from this form (see :doc:`Dynamic form generation</cookbook/dynamic_form_generation>`)
@WouterJ Collaborator
WouterJ added a note

This was an error (because it should be /cookbook/form/dynamic_form_generation). But I did removed it because it's a link to this document, I don't think that's needed. If someone finds the doc where this link should go to, I'm happy to change it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@WouterJ WouterJ referenced this pull request
Merged

[2.2] Fixed build errors #2446

@weaverryan weaverryan merged commit cd9eda4 into from
@weaverryan
Collaborator

Awesome! Removing the link makes sense to me.

@WouterJ WouterJ deleted the branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
View
11 cookbook/form/dynamic_form_modification.rst
@@ -161,12 +161,11 @@ How to Dynamically Generate Forms based on user data
====================================================
Sometimes you want a form to be generated dynamically based not only on data
-from this form (see :doc:`Dynamic form generation</cookbook/dynamic_form_generation>`)
@WouterJ Collaborator
WouterJ added a note

This was an error (because it should be /cookbook/form/dynamic_form_generation). But I did removed it because it's a link to this document, I don't think that's needed. If someone finds the doc where this link should go to, I'm happy to change it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
-but also on something else. For example depending on the user currently using
-the application. If you have a social website where a user can only message
-people who are his friends on the website, then the current user doesn't need to
-be included as a field of your form, but a "choice list" of whom to message
-should only contain users that are the current user's friends.
+from this form but also on something else. For example depending on the user
+currently using the application. If you have a social website where a user can
+only message people who are his friends on the website, then the current user
+doesn't need to be included as a field of your form, but a "choice list" of
+whom to message should only contain users that are the current user's friends.
Creating the form type
----------------------
View
2  cookbook/form/unit_testing.rst
@@ -149,7 +149,7 @@ Adding custom Extensions
------------------------
It often happens that you use some options that are added by
-:doc:`form extensions<cookbook/form/create_form_type_extension>`. One of the
+:doc:`form extensions</cookbook/form/create_form_type_extension>`. One of the
cases may be the ``ValidatorExtension`` with its ``invalid_message`` option.
The ``TypeTestCase`` loads only the core form extension so an "Invalid option"
exception will be raised if you try to use it for testing a class that depends
View
2  reference/forms/types/form.rst
@@ -19,6 +19,8 @@ on all fields.
.. include:: /reference/forms/types/options/read_only.rst.inc
+.. include:: /reference/forms/types/options/disabled.rst.inc
+
.. include:: /reference/forms/types/options/trim.rst.inc
.. include:: /reference/forms/types/options/mapped.rst.inc
Something went wrong with that request. Please try again.