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

fix: pass in maxEntries parameter to NewsFeed #39

Merged
merged 1 commit into from
Apr 3, 2024
Merged

Conversation

dealloc
Copy link
Member

@dealloc dealloc commented Apr 3, 2024

sort dispatches when storing so newest is added first

fix #38

sort dispatches when storing so newest is added first

fix #38
@dealloc dealloc added the bug Something isn't working label Apr 3, 2024
@dealloc dealloc requested a review from a team April 3, 2024 22:30
Copy link
Contributor

@chatterchats chatterchats left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything looks good on my end. Passed tests!

Copy link
Contributor

@lambstream lambstream left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A-okay for me.

@lambstream lambstream merged commit e5f5953 into master Apr 3, 2024
1 check passed
@lambstream lambstream deleted the fix/38 branch April 3, 2024 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG]: Dispatches endpoint only returning 10 or so items
3 participants