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

Pagination arrows and number links reset sort and query #37

Merged
merged 4 commits into from
Nov 10, 2016

Conversation

zberkom
Copy link
Contributor

@zberkom zberkom commented Nov 10, 2016

Closes #36

Also, updates the torch sass files to reference node_modules correctly.

@zberkom zberkom added the bug label Nov 10, 2016
@darinwilson
Copy link
Contributor

The changes look good, but I'm curious why you removed the "author" and "category" fields from the example app?

@zberkom
Copy link
Contributor Author

zberkom commented Nov 10, 2016

@darinwilson Actually, no reason in particular, I'll add them back in.

@zberkom
Copy link
Contributor Author

zberkom commented Nov 10, 2016

@darinwilson Updated

@darinwilson
Copy link
Contributor

Thanks Zach - I think it's good to keep an example of how to do associations.

@darinwilson darinwilson merged commit 27c4bbd into master Nov 10, 2016
@darinwilson darinwilson deleted the bugfix/36/pagination-arrows-reset-sort branch November 10, 2016 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants