Skip to content

feat(aci): Add pagination and event ID links to open period list#107007

Merged
malwilley merged 2 commits intomasterfrom
malwilley/feat/add-event-id-links-in-open-period-list
Jan 27, 2026
Merged

feat(aci): Add pagination and event ID links to open period list#107007
malwilley merged 2 commits intomasterfrom
malwilley/feat/add-event-id-links-in-open-period-list

Conversation

@malwilley
Copy link
Member

Closes ISWF-1929
Closes ISWF-1932

Before:

CleanShot 2026-01-26 at 13 38 58

After:

CleanShot 2026-01-26 at 13 38 38

@malwilley malwilley requested review from a team as code owners January 26, 2026 21:40
@malwilley malwilley changed the title feat(aci): Add event ID links in open period list feat(aci): Add pagination and event ID links to open period list Jan 26, 2026
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 26, 2026
/**
* Which type of pagination implementation is used by the API.
*/
paginatorType?: 'offset' | 'cursor';
Copy link
Member Author

Choose a reason for hiding this comment

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

The pagination caption was wrong because the open period endpoint uses an offset paginator, but the events endpoint uses cursor pagination. Adding this allows us to support both.

Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@malwilley malwilley force-pushed the malwilley/feat/add-event-id-links-in-open-period-list branch from f7ca39d to 2809956 Compare January 26, 2026 21:49
Copy link
Member

@ceorourke ceorourke left a comment

Choose a reason for hiding this comment

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

Looks good to my backend eyes, will not be offended if you re-request an FE reviewer

@malwilley malwilley merged commit 6d919c5 into master Jan 27, 2026
53 of 54 checks passed
@malwilley malwilley deleted the malwilley/feat/add-event-id-links-in-open-period-list branch January 27, 2026 19:04
@github-actions github-actions bot locked and limited conversation to collaborators Feb 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants