GitHub Issue 811: App permissions page shows deactivated users in group members list (#1963)

### version 7.26.3
*Released*: 31 March 2026
- GitHub Issue 811: App permissions page shows deactivated users in group members list
  - Don't show inactive users in members list for group details on permissions page
  - include "Inactive User: " prefix in member button on group management page to match what we do for permissions page

Author: cnathe

packages/components/package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@labkey/components",
-  "version": "7.26.2",
+  "version": "7.26.3",
   "description": "Components, models, actions, and utility functions for LabKey applications and pages",
   "sideEffects": false,
   "files": [

packages/components/package.json

@@ -1,6 +1,12 @@
 # @labkey/components
 Components, models, actions, and utility functions for LabKey applications and pages
 
+### version 7.26.3
+*Released*: 31 March 2026
+- GitHub Issue 811: App permissions page shows deactivated users in group members list
+  - Don't show inactive users in members list for group details on permissions page
+  - include "Inactive User: " prefix in member button on group management page to match what we do for permissions page
+
 ### version 7.26.2
 *Released*: 31 March 2026
 - GitHub Issue 919: App grid column header menu renders behind modal when grid shown in modal (more related to GitHub Issue 710)

packages/components/releaseNotes/components.md

@@ -33,20 +33,31 @@ describe('Administration actions', () => {
                 userDisplayName: 'rosalinep',
                 userEmail: 'rosalinep@labkey.com',
                 userId: 1005,
+                userActive: true,
             },
             {
                 groupId: 1035,
                 groupName: 'NewSiteGroup',
                 userDisplayName: 'rosalinep',
                 userEmail: 'rosalinep@labkey.com',
                 userId: 1005,
+                userActive: true,
+            },
+            {
+                groupId: 1035,
+                groupName: 'NewSiteGroup',
+                userDisplayName: 'inactiveUser',
+                userEmail: 'inactive@labkey.com',
+                userId: 1006,
+                userActive: false,
             },
             {
                 groupId: 1064,
                 groupName: 'group1',
                 userDisplayName: 'rosalinep',
                 userEmail: 'rosalinep@labkey.com',
                 userId: 1005,
+                userActive: true,
             },
             {
                 groupId: 1064,
@@ -68,10 +79,17 @@ describe('Administration actions', () => {
             '1035': {
                 groupName: 'NewSiteGroup',
                 members: [
+                    {
+                        id: 1006,
+                        name: 'inactive@labkey.com (inactiveUser)',
+                        type: MemberType.user,
+                        userActive: false,
+                    },
                     {
                         id: 1005,
                         name: 'rosalinep@labkey.com (rosalinep)',
                         type: MemberType.user,
+                        userActive: true,
                     },
                 ],
                 type: MemberType.siteGroup,
@@ -83,11 +101,13 @@ describe('Administration actions', () => {
                         id: 1066,
                         name: 'group2',
                         type: MemberType.group,
+                        userActive: undefined,
                     },
                     {
                         id: 1005,
                         name: 'rosalinep@labkey.com (rosalinep)',
                         type: MemberType.user,
+                        userActive: true,
                     },
                 ],
                 type: MemberType.group,

packages/components/src/internal/components/administration/actions.test.ts

@@ -21,7 +21,7 @@ import { GroupMembership, Groups, Member, MemberType } from './models';
 // Where the members array is sorted by type, and then by name. The types stand for 'group,' 'site group,' and 'user'
 export const getGroupMembership = (groups: FetchedGroup[], groupMemberships: GroupMembership[]): Groups => {
     const groupsWithMembers = groupMemberships.reduce<Groups>((memberships, groupMembership) => {
-        const { groupId, groupName, userDisplayName, userId, userEmail } = groupMembership;
+        const { groupId, groupName, userDisplayName, userId, userEmail, userActive } = groupMembership;
         if (groupId === -1) {
             return memberships;
         }
@@ -41,6 +41,7 @@ export const getGroupMembership = (groups: FetchedGroup[], groupMemberships: Gro
             name: memberIsGroup ? foundGroup.name : `${userEmail} (${userDisplayName})`,
             id: userId,
             type: memberIsGroup ? MemberType.group : MemberType.user,
+            userActive: memberIsGroup ? undefined : userActive,
         };
 
         if (groupId in memberships) {

packages/components/src/internal/components/administration/actions.ts

@@ -2,6 +2,7 @@ export interface Member {
     id: number;
     name: string;
     type: string;
+    userActive?: boolean;
 }
 
 interface Group {
@@ -13,6 +14,7 @@ interface Group {
 export type GroupMembership = {
     groupId: number;
     groupName: string;
+    userActive?: boolean;
     userDisplayName: string;
     userEmail: string;
     userId: number;