feat: resolving review comments

Signed-off-by: Arnab Dutta <arnab.bdutta@gmail.com>

demos/jans-tarp/src/options/oidcClients.tsx

@@ -88,16 +88,12 @@ function Row(props: { row: ReturnType<typeof createData>, notifyOnDataChange })
                 chrome.storage.local.set({ oidcClients: clientArr.filter(obj => obj.clientId !== row.clientId) });
             }
         });
-        handleNotifyOnDataChange();
-    }
-
-    const handleNotifyOnDataChange = () => {
         notifyOnDataChange();
-    };
+    }
 
     return (
         <React.Fragment>
-            <AuthFlowInputs isOpen={open} handleDialog={handleDialog} client={row} notifyOnDataChange={handleNotifyOnDataChange}/>
+            <AuthFlowInputs isOpen={open} handleDialog={handleDialog} client={row} notifyOnDataChange={notifyOnDataChange}/>
             <TableRow sx={{ '& > *': { borderBottom: 'unset' } }}>
                 <TableCell>
                     <Tooltip title="Delete Client from jans-tarp">
@@ -145,10 +141,6 @@ export default function OIDCClients({ data, notifyOnDataChange }) {
         setDrawerOpen(isOpen);
     };
 
-    const handleNotifyOnDataChange = () => {
-        notifyOnDataChange();
-    };
-
     return (
         <Container maxWidth="lg">
             <RegisterClient isOpen={modelOpen} handleDialog={handleDialog} />
@@ -172,9 +164,9 @@ export default function OIDCClients({ data, notifyOnDataChange }) {
                             </TableRow>
                         </TableHead>
                         <TableBody>
-                            {(data === undefined || data.length == 0) ?
+                            {(data === undefined || data?.length == 0) ?
                                 <TableCell colSpan={6}><Alert severity="warning">No Records to show.</Alert></TableCell> :
-                                data.map((row) => (<Row key={row.clientId} row={row} notifyOnDataChange={handleNotifyOnDataChange} />))
+                                data.map((row) => (<Row key={row?.clientId} row={row} notifyOnDataChange={notifyOnDataChange} />))
                             }
                         </TableBody>
                     </Table>

demos/jans-tarp/src/options/userDetails.tsx

@@ -62,7 +62,7 @@ const UserDetails = ({data, notifyOnDataChange}) => {
             </div>
             <legend><span className="number">O</span> User Details:</legend>
             <hr />
-            {data.displayToken ?
+            {data?.displayToken ?
                 <>
                     <div className="alert alert-success alert-dismissable fade in">
                         <strong>Access Token</strong>