Skip to content

Commit 8ed5006

Browse files
TCMeldrumgithub-actions[bot]
authored andcommitted
fix: use prefered name in FAP panels
1 parent 30f3cd2 commit 8ed5006

5 files changed

Lines changed: 22 additions & 5 deletions

File tree

apps/frontend/src/components/fap/Members/FapMembers.tsx

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ import { UserContext } from 'context/UserContextProvider';
2828
import { BasicUserDetails, UserRole, Fap } from 'generated/sdk';
2929
import { useCheckAccess } from 'hooks/common/useCheckAccess';
3030
import { useRenewToken } from 'hooks/common/useRenewToken';
31-
import { useFapReviewersData } from 'hooks/fap/useFapReviewersData';
31+
import { FapMember, useFapReviewersData } from 'hooks/fap/useFapReviewersData';
3232
import { tableIcons } from 'utils/materialIcons';
3333
import useDataApiWithFeedback from 'utils/useDataApiWithFeedback';
3434
import { getFullUserName } from 'utils/user';
@@ -43,7 +43,13 @@ type FapMembersProps = {
4343
};
4444

4545
const columns = [
46-
{ title: 'Name', field: 'user.firstname' },
46+
{
47+
title: 'Name',
48+
render: (rowData: FapMember) =>
49+
rowData.user.preferredname
50+
? rowData.user.preferredname
51+
: rowData.user.firstname,
52+
},
4753
{
4854
title: 'Surname',
4955
field: 'user.lastname',

apps/frontend/src/components/fap/Proposals/AssignFapMemberToProposalModal.tsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,11 @@ type AssignFapMemberToProposalModalProps = {
2727
const memberRole = (member: FapAssignedMember) => `${member.role?.title}`;
2828

2929
const columns = [
30-
{ title: 'Name', field: 'firstname' },
30+
{
31+
title: 'Name',
32+
render: (rowData: FapAssignedMember) =>
33+
rowData.preferredname ? rowData.preferredname : rowData.firstname,
34+
},
3135
{ title: 'Surname', field: 'lastname' },
3236
{ title: 'Proposal Count', field: 'proposalsCountByCall' },
3337
{

apps/frontend/src/components/fap/Proposals/FapAssignedReviewersTable.tsx

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,12 @@ const FapAssignedReviewersTable = ({
106106
const assignmentColumns = [
107107
{
108108
title: 'First name',
109-
field: 'user.firstname',
109+
render: (rowData: FapProposalAssignmentType) =>
110+
rowData.user
111+
? rowData.user.preferredname
112+
? rowData.user.preferredname
113+
: rowData.user.firstname
114+
: '-',
110115
},
111116
{
112117
title: 'Last name',

apps/frontend/src/components/fap/Proposals/MultiRankAssignmentDialog.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ export const MultiRankAssignmentDialog = ({
9494
}}
9595
>
9696
<TableCell component="th" scope="row">
97-
{row.firstname}
97+
{row.preferredname ? row.preferredname : row.firstname}
9898
</TableCell>
9999
<TableCell align="left">{row.lastname}</TableCell>
100100
<TableCell align="right">

apps/frontend/src/components/user/PeopleTable.tsx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -325,6 +325,7 @@ const PeopleTable = ({
325325
...(selectedItems.map((selectedItem) => ({
326326
id: selectedItem.id,
327327
firstname: selectedItem.firstname,
328+
preferredname: selectedItem.preferredname,
328329
lastname: selectedItem.lastname,
329330
institution: selectedItem.institution,
330331
institutionId: selectedItem.institutionId,
@@ -346,6 +347,7 @@ const PeopleTable = ({
346347
{
347348
id: selectedItem.id,
348349
firstname: selectedItem.firstname,
350+
preferredname: selectedItem.preferredname,
349351
lastname: selectedItem.lastname,
350352
institution: selectedItem.institution,
351353
institutionId: selectedItem.institutionId,

0 commit comments

Comments
 (0)