Skip to content
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

Feature post orderby #1183

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Feature post orderby #1183

wants to merge 5 commits into from

Conversation

tox2ik
Copy link

@tox2ik tox2ik commented Mar 14, 2014

Please read the message for e0368a4.

This patch covers this feature request: #272
Note that I'm using $.extend to append pagination and sorting settings, because after trying the sample code in that request, I discovered that other settings would get lost.

The default behavior of appending ?order_by=... to the url is inconsistent with
this.options.ajaxSettings.type when it is set to POST.

With this patch, a back-end service can be configured to ignore GET parameters
all together, which could be good in some cases.

fixes: #272
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

Successfully merging this pull request may close these issues.

1 participant