diff --git a/site/src/components/Resources/AgentMetadata.stories.tsx b/site/src/components/Resources/AgentMetadata.stories.tsx index b5159a5830db2..9e5bedfd907e9 100644 --- a/site/src/components/Resources/AgentMetadata.stories.tsx +++ b/site/src/components/Resources/AgentMetadata.stories.tsx @@ -57,7 +57,7 @@ Example.args = { { result: { ...resultDefaults, - value: "cant see it", + value: "stale value", age: 300, }, description: { @@ -76,18 +76,7 @@ Example.args = { description: { ...descriptionDefaults, display_name: "Error", - }, - }, - { - result: { - ...resultDefaults, - value: "oops", - error: "fatal error", - }, - description: { - ...descriptionDefaults, - display_name: "Error", - key: "stale", + key: "error", }, }, { diff --git a/site/src/components/Resources/AgentMetadata.tsx b/site/src/components/Resources/AgentMetadata.tsx index d72d70ab7efe0..d8f35c3faec22 100644 --- a/site/src/components/Resources/AgentMetadata.tsx +++ b/site/src/components/Resources/AgentMetadata.tsx @@ -6,6 +6,8 @@ import dayjs from "dayjs" import { createContext, FC, useContext, useEffect, useState } from "react" import Skeleton from "@mui/material/Skeleton" import { MONOSPACE_FONT_FAMILY } from "theme/constants" +import { combineClasses } from "utils/combineClasses" +import Tooltip from "@mui/material/Tooltip" type ItemStatus = "stale" | "valid" | "loading" @@ -44,22 +46,33 @@ const MetadataItem: FC<{ item: WorkspaceAgentMetadata }> = ({ item }) => { // users that what's shown is real. If times aren't correctly synced this // could be buggy. But, how common is that anyways? const value = - status === "stale" || status === "loading" ? ( + status === "loading" ? ( + ) : status === "stale" ? ( + +
+ {item.result.value} +
+
) : (
{item.result.value}
@@ -226,6 +239,11 @@ const useStyles = makeStyles((theme) => ({ color: theme.palette.error.main, }, + metadataStale: { + color: theme.palette.text.disabled, + cursor: "pointer", + }, + skeleton: { marginTop: theme.spacing(0.5), },