SP67 Fix incorrect auth profile owner source (#790)

* Fix incorrect auth profile owner source * Fix heading, default to user when making new auth profile * Fix incorrect owner assignment * Add ability to set sort key --------- Co-authored-by: ikethecoder <ikethecoder@copeconsulting.ca>
bcgov · May 17, 2023 · e011b1c · e011b1c
1 parent bbc11e6
commit e011b1c
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 3 deletions.
diff --git a/src/nextapp/components/authorization-profile-form/authorization-profile-dialog.tsx b/src/nextapp/components/authorization-profile-form/authorization-profile-dialog.tsx
@@ -270,6 +270,7 @@ const AuthorizationProfileDialog: React.FC<AuthorizationProfileDialogProps> = ({
     },
     [data, handleCreateProfile, handleSaveProfile, id]
   );
+  const owner = id ? data?.owner : data?.owner ?? user;
 
   React.useEffect(() => {
     setName(data?.name);
@@ -305,7 +306,7 @@ const AuthorizationProfileDialog: React.FC<AuthorizationProfileDialogProps> = ({
                   </Tab>
                   <Box flex={1} />
                   <Box p="relative" mt={-10} mb={2}>
-                    <UserProfile data={user} />
+                    <UserProfile heading="Owner" data={owner} />
                   </Box>
                 </TabList>
               </Tabs>

diff --git a/src/nextapp/components/table/table.tsx b/src/nextapp/components/table/table.tsx
@@ -22,6 +22,7 @@ interface Column extends TableColumnHeaderProps {
   name: React.ReactNode;
   key?: string;
   sortable?: boolean;
+  sortKey?: string;
 }
 
 interface ApsTableProps extends TableProps {
@@ -109,7 +110,7 @@ const ApsTable: React.FC<ApsTableProps> = ({
                 }
                 onClick={
                   sort && tableProps.key
-                    ? handleSort(tableProps.key)
+                    ? handleSort(tableProps.sortKey ?? tableProps.key)
                     : undefined
                 }
                 _hover={{

diff --git a/src/nextapp/components/user-profile/user-profile.tsx b/src/nextapp/components/user-profile/user-profile.tsx
@@ -26,6 +26,10 @@ const UserProfile: React.FC<UserProfileProps> = ({
   ...props
 }) => {
   const testId = props['data-testid'] ?? 'user-profile';
+  if (!data) {
+    return null;
+  }
+
   return (
     <Box {...props}>
       <Heading size="sm" mb={3.5}>

diff --git a/src/nextapp/pages/manager/consumers/index.tsx b/src/nextapp/pages/manager/consumers/index.tsx
@@ -248,7 +248,7 @@ const ConsumersPage: React.FC<
                 key: 'tags',
                 sortable: false,
               },
-              { name: 'Updated', key: 'sortDate' },
+              { name: 'Updated', key: 'sortDate', sortKey: 'updatedAt' },
             ]}
             data={consumers}
             data-testid="all-consumer-control-tbl"