TUI-92 - Streams in Dashboard #178
Conversation
| @@ -107,6 +109,14 @@ const Dashboard: React.FC = () => { | |||
| counter={`${jobs?.data?.result?.length} jobs`} | |||
| loading={jobs?.isLoading} | |||
| /> | |||
| <DashboardCard | |||
| icon="project" | |||
| name="Projects" | |||
There was a problem hiding this comment.
Can you rename this to Streams? Since that is the top level name of the service.
There was a problem hiding this comment.
Streams is the top level name this group of services falls into, but this tile displays the number of Projects. The UI should specify that this is what it is showing, saying it shows the number of streams would not make sense.
There was a problem hiding this comment.
We have other uses for the word "Projects" in future downstream consumers of tapis-ui. This card should have Streams as a title but the text can remain 100 projects. In theory, this could be built out to have more detail, like 100 projects, 20 instruments etc.
There was a problem hiding this comment.
Furthermore, the dashboard cards and side nav consistently represent the top level services.
| @@ -107,6 +109,14 @@ const Dashboard: React.FC = () => { | |||
| counter={`${jobs?.data?.result?.length} jobs`} | |||
| loading={jobs?.isLoading} | |||
| /> | |||
| <DashboardCard | |||
| icon="project" | |||
| name="Projects" | |||
There was a problem hiding this comment.
We have other uses for the word "Projects" in future downstream consumers of tapis-ui. This card should have Streams as a title but the text can remain 100 projects. In theory, this could be built out to have more detail, like 100 projects, 20 instruments etc.
| @@ -107,6 +109,14 @@ const Dashboard: React.FC = () => { | |||
| counter={`${jobs?.data?.result?.length} jobs`} | |||
| loading={jobs?.isLoading} | |||
| /> | |||
| <DashboardCard | |||
| icon="project" | |||
| name="Projects" | |||
There was a problem hiding this comment.
Furthermore, the dashboard cards and side nav consistently represent the top level services.
Overview:
Add projects listing to dashboard
Related Github Issues:
Summary of Changes:
Testing Steps:
UI Photos:
Notes:
Dashboard link not implemented, will be implemented in further pr