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

Updated activities endpoint to better support client-side pagination #44

Merged
merged 1 commit into from
Sep 24, 2024

Conversation

mike182uk
Copy link
Member

@mike182uk mike182uk commented Sep 19, 2024

no refs

In order for client-side pagination to work effectively, the activities endpoint has been updated to:

  • Handle filtering required on the client-side (was previously done client-side)
  • Return nested replies in returned activities
  • Expand inReplyTo to full object

Verified

This commit was signed with the committer’s verified signature.
mike182uk Michael Barrett
no refs

In order for client-side pagination to work effectively, the activities endpoint
has been updated to:

- Handle filtering required on the client-side (was previously done client-side)
- Return nested replies in returned activities
- Expand `inReplyTo` to full object
@mike182uk mike182uk merged commit 294a05e into main Sep 24, 2024
1 check passed
@mike182uk mike182uk deleted the clientside-pagination-prep branch September 24, 2024 09:46
mike182uk added a commit to TryGhost/Ghost that referenced this pull request Sep 24, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
refs
[TryGhost/ActivityPub#44](TryGhost/ActivityPub#44)

To support pagination in the activitypub app, the following changes have
been made:

- Move filtering and sorting of activities to the server
- Refactor how comments are processed (comments are now returned as part
of the activity)
- Refactor how replies to own activities are processed in the activities
tab (removed object map)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants