Skip to content

feat: implement ListActivities pagination#5468

Merged
boojack merged 3 commits intousememos:mainfrom
biplavbarua:feat/activity-pagination
Mar 4, 2026
Merged

feat: implement ListActivities pagination#5468
boojack merged 3 commits intousememos:mainfrom
biplavbarua:feat/activity-pagination

Conversation

@biplavbarua
Copy link
Contributor

This PR implements pagination for the ListActivities API using page_size and page_token.

Changes

  • Store: Added Limit and Offset to FindActivity struct.
  • Drivers: Updated SQLite, MySQL, and PostgreSQL drivers to support pagination queries.
  • Service: Updated ListActivities in activity_service.go to handle pagination parameters and generate next_page_token.
  • Tests: Added activity_service_test.go to verify pagination logic.

Verification

  • Added new unit test TestListActivities covering creation and pagination flows.
  • Verified locally with go test.

@boojack boojack force-pushed the main branch 2 times, most recently from a196c72 to 71e8a06 Compare February 10, 2026 01:15
@boojack boojack self-requested a review as a code owner February 24, 2026 14:23
@boojack boojack force-pushed the main branch 2 times, most recently from 2b6b987 to 47d9414 Compare February 24, 2026 14:55
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Member

@boojack boojack left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the contribution!

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@boojack boojack merged commit def1232 into usememos:main Mar 4, 2026
5 checks passed
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.

2 participants