feat: add frontend for app statuses #17178
Conversation
| }, | ||
| }; | ||
|
|
||
| const formatURI = (uri: string) => { |
There was a problem hiding this comment.
There's a lot of util-esque functionality in here I think we should eventually extract (like the short URIs), but don't want that to block right now.
| const agent = agents.find((agent) => agent.id === status.agent_id); | ||
|
|
||
| if (currentApp && agent) { | ||
| const appSlug = currentApp.slug || currentApp.display_name; |
There was a problem hiding this comment.
In what context would we derive the slug from the url encoded display name?
There was a problem hiding this comment.
This is just copying code from how it's done before, which is weird, but I guess for backwards compat?
| : theme.palette.text.disabled; | ||
| switch (state) { | ||
| case "complete": | ||
| return <CheckCircle sx={{ color, fontSize: 18 }} />; |
There was a problem hiding this comment.
I'm not in love with these icons but I get this is an experiment.
| } | ||
|
|
||
| // Determine if app link should be shown | ||
| const showAppLink = |
There was a problem hiding this comment.
Took me awhile to get the avoiding redundant links aspect of this, a comment would go a long way here I think.
There was a problem hiding this comment.
Will do after merge since we're in a crunch.
Check out the stories for the exacts...