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
Adding limit check for demo #3357
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good I think, only ffound a weird indent in the tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I will test and we can release tomorrow if it passes.
Export is returning all posts, but I think that was by design , right? The checklist passed. |
This pull request makes the following changes:
Test checklist:
For deployment with 25+ posts you should only ever see the first 25
You should not be able to load more
You should be able to see that you are viewing 25 of real number of posts
You should not be able to get more posts by using the offset
I certify that I ran my checklist
Fixes ushahidi/platform# .
Ping @ushahidi/platform