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

<link> to the first page of ATOM feeds #186

Merged
merged 1 commit into from
Apr 8, 2017
Merged

Conversation

divergentdave
Copy link
Collaborator

Inspired by #185. Since the number of reports per page is different between HTML and ATOM pages, it doesn't make sense for page 400 of the HTML ​search results to link to page 400 of the ATOM feed.

This still needs some testing

@divergentdave divergentdave merged commit baa6852 into master Apr 8, 2017
@divergentdave divergentdave deleted the atom-link-first-page branch April 8, 2017 01:46
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