diff --git a/app/components/Header.tsx b/app/components/Header.tsx index f02d6e3..e166c7f 100644 --- a/app/components/Header.tsx +++ b/app/components/Header.tsx @@ -107,7 +107,7 @@ export default function Header(data: Props) { {data.user.picture ? ( {data.user.name} ) : ( @@ -131,7 +131,7 @@ export default function Header(data: Props) {
-

{data.user.name}

+

{data.user.name || data.user.displayName}

{data.user.email}

diff --git a/app/routes/machines/components/machine-row.tsx b/app/routes/machines/components/machine-row.tsx index c078773..4513af2 100644 --- a/app/routes/machines/components/machine-row.tsx +++ b/app/routes/machines/components/machine-row.tsx @@ -64,7 +64,7 @@ export default function MachineRow({ {node.givenName}

- {node.user.name || node.user.email || node.user.id} + {node.user.name || node.user.displayName || node.user.email || node.user.id}

{mapTagsToComponents(node, uiTags)} diff --git a/app/routes/machines/dialogs/move.tsx b/app/routes/machines/dialogs/move.tsx index 305a358..3b86bb7 100644 --- a/app/routes/machines/dialogs/move.tsx +++ b/app/routes/machines/dialogs/move.tsx @@ -34,7 +34,7 @@ export default function Move({ machine, users, isOpen, setIsOpen }: MoveProps) { }} > {users.map((user) => ( - {user.name || user.email || user.id} + {user.name || user.displayName || user.email || user.id} ))} diff --git a/app/routes/machines/dialogs/new.tsx b/app/routes/machines/dialogs/new.tsx index 35c6320..df98129 100644 --- a/app/routes/machines/dialogs/new.tsx +++ b/app/routes/machines/dialogs/new.tsx @@ -46,7 +46,7 @@ export default function NewMachine(data: NewMachineProps) { placeholder="Select a user" > {data.users.map((user) => ( - {user.name || user.email || user.id} + {user.name || user.displayName || user.email || user.id} ))} diff --git a/app/routes/machines/machine.tsx b/app/routes/machines/machine.tsx index a081085..051794f 100644 --- a/app/routes/machines/machine.tsx +++ b/app/routes/machines/machine.tsx @@ -109,7 +109,7 @@ export default function Page() {
- {node.user.name || node.user.email || node.user.id} + {node.user.name || node.user.displayName || node.user.email || node.user.id}
@@ -254,7 +254,7 @@ export default function Page() { className="w-full max-w-full grid grid-cols-1 lg:grid-cols-2 gap-y-2 sm:gap-x-12" >
- + - + diff --git a/app/routes/settings/auth-keys/dialogs/add-auth-key.tsx b/app/routes/settings/auth-keys/dialogs/add-auth-key.tsx index faa3753..18df724 100644 --- a/app/routes/settings/auth-keys/dialogs/add-auth-key.tsx +++ b/app/routes/settings/auth-keys/dialogs/add-auth-key.tsx @@ -36,7 +36,7 @@ export default function AddAuthKey(data: AddAuthKeyProps) { }} > {data.users.map((user) => ( - {user.name || user.email || user.id} + {user.name || user.displayName || user.email || user.id} ))} All, ...keys.map(({ user }) => ( - {user.name || user.email || user.id} + {user.name || user.displayName || user.email || user.id} )), ]} diff --git a/app/routes/users/components/user-row.tsx b/app/routes/users/components/user-row.tsx index b6d9118..ce8d531 100644 --- a/app/routes/users/components/user-row.tsx +++ b/app/routes/users/components/user-row.tsx @@ -26,14 +26,14 @@ export default function UserRow({ user, role }: UserRowProps) { {user.profilePicUrl ? ( {user.name} ) : ( )}
-

{user.name}

+

{user.name || user.displayName}

{user.email}

diff --git a/app/routes/users/dialogs/delete-user.tsx b/app/routes/users/dialogs/delete-user.tsx index 0e2aae2..2490adb 100644 --- a/app/routes/users/dialogs/delete-user.tsx +++ b/app/routes/users/dialogs/delete-user.tsx @@ -8,7 +8,7 @@ interface DeleteProps { } export default function DeleteUser({ user, isOpen, setIsOpen }: DeleteProps) { - const name = user.name; + const name = user.name || user.displayName; return ( diff --git a/app/routes/users/dialogs/reassign-user.tsx b/app/routes/users/dialogs/reassign-user.tsx index d31fdcc..ab88195 100644 --- a/app/routes/users/dialogs/reassign-user.tsx +++ b/app/routes/users/dialogs/reassign-user.tsx @@ -21,7 +21,7 @@ export default function ReassignUser({ - Change role for {user.name}? + Change role for {user.name || user.displayName}? Most roles are carried straight from Tailscale. However, keep in mind that I have not fully implemented permissions yet and some things may diff --git a/app/routes/users/dialogs/rename-user.tsx b/app/routes/users/dialogs/rename-user.tsx index a18b830..b47245b 100644 --- a/app/routes/users/dialogs/rename-user.tsx +++ b/app/routes/users/dialogs/rename-user.tsx @@ -13,9 +13,9 @@ export default function RenameUser({ user, isOpen, setIsOpen }: RenameProps) { return ( - Rename {user.name}? + Rename {user.name || user.displayName}? - Enter a new username for {user.name}. Changing a username will not + Enter a new username for {user.name || user.displayName}. Changing a username will not update any ACL policies that may refer to this user by their old username. diff --git a/app/utils/oidc.ts b/app/utils/oidc.ts index 113bb09..788029c 100644 --- a/app/utils/oidc.ts +++ b/app/utils/oidc.ts @@ -132,10 +132,6 @@ function getName(user: client.UserInfoResponse, claims: client.IDToken) { return user.name; } - if (user.displayName) { - return user.displayName; - } - if (claims.name && typeof claims.name === 'string') { return claims.name; }