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

OrderableListMixin docs #191

Closed
epicserve opened this issue Aug 6, 2015 · 4 comments
Closed

OrderableListMixin docs #191

epicserve opened this issue Aug 6, 2015 · 4 comments

Comments

@epicserve
Copy link
Contributor

So I was reading the docs for OrderableListMixin and I wasn't seeing anything about it adding anything to the context_data with variables for order_by and ordering. I just checked the source code however and it looks like the mixin does add to the context, so is this just an oversight as to the reason it's not in the docs? If so, would you like me to give you a pull request with that info in the docs?

@kennethlove
Copy link
Member

Always submit a PR 😸

@epicserve
Copy link
Contributor Author

Would it be good to include an example of how you might use this information if you're using something like bootstrap or is that overkill?

@kennethlove
Copy link
Member

Sure, so long as the example isn't ridiculously over-blown.

@chrisjones-brack3t
Copy link
Member

closing very old issues

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