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

Only display 30 entries now? #33

Closed
coodoo opened this issue Jul 27, 2014 · 4 comments
Closed

Only display 30 entries now? #33

coodoo opened this issue Jul 27, 2014 · 4 comments
Assignees
Labels

Comments

@coodoo
Copy link

coodoo commented Jul 27, 2014

Seems starting a couple days ago it will only display 30 instead of 60 entries at a time, is something changed?

@cheeaun cheeaun added the bug label Jul 27, 2014
@cheeaun cheeaun self-assigned this Jul 27, 2014
@cheeaun
Copy link
Owner

cheeaun commented Jul 27, 2014

Fixed in cheeaun/node-hnapi@9db9ab7

Looks like HN now supports pagination 😂

@cheeaun cheeaun closed this as completed Jul 27, 2014
@coodoo
Copy link
Author

coodoo commented Jul 27, 2014

Thanks for the quick fix, great works!

Btw, is there any reason just to display 60 entries instead of showing more pages?

@cheeaun
Copy link
Owner

cheeaun commented Jul 27, 2014

@coodoo as for now, it'll be just 60. The main reason is to reduce API calls that will scrape the HN web site. It's not difficult to add pagination to the API now, but I don't want people to abuse it.

@coodoo
Copy link
Author

coodoo commented Jul 28, 2014

Understood, thx.

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

No branches or pull requests

2 participants