Project

General

Profile

Actions

Bug #21249

closed

Paging doesn't work for group members and permissions tables

Added by Peter Amstutz 3 months ago. Updated 3 days ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Workbench2
Story points:
-
Release relationship:
Auto

Description

On the group panel, the "members" and "permission" tabs, if there is more than 1 page of items, using the forward/backward buttons will change the numerical range but not actually change what is displayed in the table.


Subtasks 1 (0 open1 closed)

Task #21342: Review 21249-group-pagingResolvedStephen Smith02/27/2024Actions
Actions #1

Updated by Peter Amstutz 3 months ago

  • Description updated (diff)
Actions #2

Updated by Peter Amstutz 3 months ago

  • Target version changed from Development 2024-01-17 sprint to Development 2024-01-31 sprint
Actions #3

Updated by Peter Amstutz 3 months ago

  • Target version changed from Development 2024-01-31 sprint to Development 2024-01-17 sprint
Actions #4

Updated by Peter Amstutz about 2 months ago

  • Assigned To set to Stephen Smith
Actions #5

Updated by Peter Amstutz about 2 months ago

  • Target version changed from Development 2024-01-17 sprint to Development 2024-01-31 sprint
Actions #6

Updated by Peter Amstutz about 2 months ago

Also noted: the listing group membership seems buggy. In once case, observed that user entries were duplicated, in another case, the entire user list appeared instead of just the members.

Actions #7

Updated by Peter Amstutz about 2 months ago

  • Assigned To deleted (Stephen Smith)
Actions #8

Updated by Peter Amstutz about 2 months ago

  • Assigned To set to Stephen Smith
Actions #9

Updated by Peter Amstutz about 1 month ago

  • Release set to 69
Actions #10

Updated by Peter Amstutz about 1 month ago

  • Target version changed from Development 2024-01-31 sprint to Development 2024-02-14 sprint
Actions #11

Updated by Peter Amstutz about 1 month ago

  • Assigned To deleted (Stephen Smith)
Actions #12

Updated by Peter Amstutz about 1 month ago

  • Assigned To set to Stephen Smith
Actions #13

Updated by Peter Amstutz 18 days ago

  • Target version changed from Development 2024-02-14 sprint to Development 2024-02-28 sprint
Actions #14

Updated by Stephen Smith 4 days ago

  • Status changed from New to In Progress
Actions #15

Updated by Stephen Smith 4 days ago

Changes at arvados|bba33fe67b5126c092ac09e6fe0ff388aefd5619
Tests developer-run-tests-services-workbench2: #509

  • All agreed upon points are implemented / addressed.
    • Added missing pagination to group members / permissions middlewares
    • Added unit tests to check for correct pagination filters
  • Anything not implemented (discovered or discussed during work) has a follow-up story.
    • n/a
  • Code is tested and passing, both automated and manual, what manual testing was done is described
    • automated & manually checking
  • Documentation has been updated.
    • n/a
  • Behaves appropriately at the intended scale (describe intended scale).
    • n/a
  • Considered backwards and forwards compatibility issues between client and server.
    • n/a
  • Follows our coding standards and GUI style guidelines.
    • no ux changes
Actions #16

Updated by Lisa Knox 3 days ago

This all looks good, but I noticed some unusual behavior when viewing the 'All Users' group:

  1. When first viewing the group, the usernames are replaced with UUIDs and the 'Account Status' and 'Visible to other members' fields are completely blank
  2. When paging to the second page (which works now), you see the same thing but there is a pop-in a second after the initial load when all of the correct data is populated

If these are beyond the scope of this ticket, then they deserve a follow-up and this lgtm.

Actions #17

Updated by Stephen Smith 3 days ago

  • Status changed from In Progress to Resolved

Created new ticket #21550 for loading indicators on status and visible fields

Merged in arvados|6f3e2cd586172574f2b6ee0f03fc5c79e7ce3b32

Actions

Also available in: Atom PDF