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.

Sign up for GitHub

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

Jump to bottom

Support limit parameter #14

Closed
simonft opened this issue May 13, 2023 · 0 comments · Fixed by #18
Closed

Support limit parameter #14

simonft opened this issue May 13, 2023 · 0 comments · Fixed by #18

Comments

@simonft
Copy link
Contributor

simonft commented May 13, 2023

The spec shows a limit parameter being available, which should default to 50. As best as I can tell this repo doesn't support it, and always returns all the posts. This seems especially odd since it does have a cursor parameter, but it's never needed if the API always returns all the posts since the beginning of time.
simonft added a commit to simonft/feed-generator that referenced this issue May 14, 2023
@simonft
Use limit parameter in post query
6f1d41f 
Fixes bluesky-social#14
@simonft simonft mentioned this issue May 14, 2023
Merged
dholms pushed a commit that referenced this issue May 15, 2023
@simonft
Use limit parameter in post query (#18)
d93c993 
Fixes #14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use limit parameter in post query simonft/feed-generator
1 participant
@simonft