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

Draft list paging #356

Draft
wants to merge 2 commits into
base: develop
from
Draft

Draft list paging #356

wants to merge 2 commits into from

Conversation

@thebaer
Copy link
Member

thebaer commented Jul 30, 2020

This loads 10 posts at a time on a user's Drafts page, and includes a "Load more" button at the bottom to dynamically load more entries (T401, T696)

It also modifies the GET /api/me/posts endpoint to accept the following parameters:

  • anonymous=1 - returns only anonymous / draft posts
  • page={number}- returns posts for the given page {number}. If 0, all posts are returned.

  • I have signed the CLA
thebaer added 2 commits Jul 30, 2020
This adds a "load more" button to the bottom of the draft posts page,
which calls /api/me/posts with new parameters and the current page
number. It then populates the page accordingly.

Ref T696 - load anon. posts with ?anonymous=1&page=1
Ref T401 - completes UI for post loading
@thebaer thebaer added this to the 0.13 milestone Jul 30, 2020
@thebaer
Copy link
Member Author

thebaer commented Jul 30, 2020

Last issue to fix:

  • Loaded posts don't include the "Move to..." action
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.