Merge pull request #136 from navi-sa/fix-name-use-for-manager

INFRA-3147 | replace name check to email check
This commit is contained in:
Dhruv Joshi
2024-04-10 14:28:03 +05:30
committed by GitHub

View File

@@ -29,7 +29,7 @@ const SeverityWiseList: FC<SeverityWiseListProps> = (
const Role = useAuthData();
const computedClassName = (item: Member): string => {
return (
(manager?.name === item.name && styles['manager-wrapper']) ||
(manager?.email === item.email && styles['manager-wrapper']) ||
(userEmail !== managerEmail &&
!Role.includes('Admin') &&
styles['team-member-view']) ||
@@ -108,7 +108,7 @@ const SeverityWiseList: FC<SeverityWiseListProps> = (
>
<div className={styles['details-container']}>
<div className={styles['icon-wrapper']}>
{manager?.name === item.name ? (
{manager?.email === item.email ? (
<ManagerIconFill />
) : (
<div>
@@ -127,10 +127,10 @@ const SeverityWiseList: FC<SeverityWiseListProps> = (
<Typography variant="p3"> {item.name}</Typography>
<Typography variant="p5"> {item.email}</Typography>
<Typography variant="p5">
{manager?.name === item.name ? 'Manager' : ''}
{manager?.email === item.email ? 'Manager' : ''}
</Typography>{' '}
<div className={styles['make-manger-button']}>
{manager?.name !== item.name && (
{manager?.email !== item.email && (
<Button
variant="text"
onClick={() => {
@@ -142,7 +142,7 @@ const SeverityWiseList: FC<SeverityWiseListProps> = (
)}
</div>
</div>
{manager?.name !== item.name && (
{manager?.email !== item.email && (
<div
key={`${item.id}_dialog`}
onClick={() => {