Skip to content

fix: enabled pagination for data access users table - #1304

Merged
yoganandaness merged 10 commits into
developfrom
SWAP-5201-uo-onle-first-10-data-access-users-visible
Jan 15, 2026
Merged

fix: enabled pagination for data access users table#1304
yoganandaness merged 10 commits into
developfrom
SWAP-5201-uo-onle-first-10-data-access-users-visible

Conversation

@yoganandaness

@yoganandaness yoganandaness commented Jan 9, 2026

Copy link
Copy Markdown
Contributor

Description

In the Data Access users table of a Proposal, only first 10 users are visible and the remaining is hidden. This is because that the pagination has been disabled for the table.

Motivation and Context

To display all the Data Access Users.

How Has This Been Tested

Manually

Fixes

Fixes the situation where the user is unable to view all the data access users associated to a Proposal.

This change not only fix the issue in the Data access users, but also with Co-Proposers table, Visitors table.

Changes

Enabled the pagination flag

Depends on

Tests included/Docs Updated?

  • I have added tests to cover my changes.
  • All relevant doc has been updated

@yoganandaness
yoganandaness requested a review from a team as a code owner January 9, 2026 14:52
@yoganandaness
yoganandaness requested review from Scott-James-Hurley, TCMeldrum and jekabs-karklins and removed request for a team and Scott-James-Hurley January 9, 2026 14:52

@jekabs-karklins jekabs-karklins left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Excellent

yoganandaness and others added 2 commits January 13, 2026 13:20
@yoganandaness
yoganandaness enabled auto-merge (squash) January 15, 2026 14:45
@yoganandaness
yoganandaness merged commit 22e8909 into develop Jan 15, 2026
22 checks passed
@yoganandaness
yoganandaness deleted the SWAP-5201-uo-onle-first-10-data-access-users-visible branch January 15, 2026 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants