Bug #21221
closed
group panel Members count is inaccurate
Added by Peter Amstutz 9 months ago.
Updated 5 months ago.
Release relationship:
Auto
Description
On the "Groups" panel, the column listing the number of members is inaccurate.
When there are more than 100 users on the system and/or groups with more than 100 users, workbench does not load all the membership links and as a result the "member" counts are wrong.
These should probably be populated by making an API call for each group with limit=0 and looking at "items available".
- Description updated (diff)
- Target version changed from Future to Development 2024-01-17 sprint
- Target version changed from Development 2024-01-17 sprint to Development 2024-01-31 sprint
- Target version changed from Development 2024-01-31 sprint to Development 2024-01-17 sprint
- Target version changed from Development 2024-01-17 sprint to Development 2024-01-31 sprint
- Assigned To set to Lisa Knox
- Target version changed from Development 2024-01-31 sprint to Development 2024-02-14 sprint
- Assigned To deleted (
Lisa Knox)
- Target version changed from Development 2024-02-14 sprint to Development 2024-02-28 sprint
- Target version changed from Development 2024-02-28 sprint to Development 2024-03-13 sprint
- Assigned To set to Stephen Smith
- Status changed from New to In Progress
Changes at arvados|75017b1e056df82d947db19a024c524072d0d55d
Tests developer-run-tests-services-workbench2: #518 ![](https://ci.arvados.org/buildStatus/icon?job=developer-run-tests-services-workbench2&build=518)
- All agreed upon points are implemented / addressed.
- Changed groups middleware to request count of each group's members
- Count is now stored in the resource store on a local property memberCount, which is absent/undefined initially and null if the request failed
- Added a loading indicator pulser that shows while the limit=0 list request is loading, if the request fails a grey error icon shows with a tooltip saying that the count failed to load
- Added unit tests to verify that the middleware requests count and stores it on the resource, as well as stores null if the request failed
- Added unit test to verify loading pulser, error icon, and correct count on the GroupMembersCount renderer
- Anything not implemented (discovered or discussed during work) has a follow-up story.
- Code is tested and passing, both automated and manual, what manual testing was done is described
- Added unit tests for everything
- Documentation has been updated.
- Behaves appropriately at the intended scale (describe intended scale).
- Only 1 request per group on the current data explorer page
- Considered backwards and forwards compatibility issues between client and server.
- Follows our coding standards and GUI style guidelines.
This all LGTM, but the inline loading pulser is something I haven't seen anywhere else on WB2 (usually it's an entire table replaced by the loading view, not a single item) so it looks out of place to me, even though it looks good in and of itself. It would be nice to use this same pulser elsewhere going forward (where applicable) to foster some visual consistency across WB2.
- Status changed from In Progress to Resolved
Also available in: Atom
PDF