Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

pass highlighting to SolrPage Paginator pages #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tingletech
Copy link
Contributor

Mark Redar's change to sorlpy that we needed for the portal project demo

Without this, hit highlighting didn't work for paginated pages

@freddrake
Copy link
Contributor

Any chance for a test?

@edsu
Copy link
Collaborator

edsu commented Apr 7, 2015

Speaking of tests, I'm going to try to get them running on Travis.

@tingletech
Copy link
Contributor Author

sure, I'll also add a page size test to the SolrPage as seen in key/solrpy@5b5f14d ~ line 71, 98 . Would it make sense to test highlighting and page size at the same time (in the same test)?

@edsu
Copy link
Collaborator

edsu commented Apr 9, 2015

@tingletech I think any test would be great ; there are already tests that demonstrate this PR doesn't break anything. So any sort of test of the expected new behavior would be great.

@tingletech
Copy link
Contributor Author

It would also be cool to test that this even still works from django, last time I tried was django 1.3.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants