Fix docs: Cookbook form_ajax_refs definition #659
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I really don't understand it deeply but the original code example for
form_ajax_refs
from the cookbook does not work.It is not documented anywhere why, I just found the answer in
tests/test_ajax.py
.child
in theParent
class but there is achildren
relationship"order_by": ("id",)
just does not work (sqlalchemy.exc.ArgumentError: ORDER BY expression expected, got ('id',).
) but"order_by": "id"
worksIt would be also fine to add
__str__
method to the example implementation of the classChild
to be more clear about what is used in the list.