diff --git a/coderd/apidoc/docs.go b/coderd/apidoc/docs.go index 00f1b57ef7f8c..ea0979333b451 100644 --- a/coderd/apidoc/docs.go +++ b/coderd/apidoc/docs.go @@ -8091,7 +8091,6 @@ const docTemplate = `{ "single_tailnet", "template_restart_requirement", "deployment_health_page", - "template_parameters_insights", "workspaces_batch_actions" ], "x-enum-varnames": [ @@ -8101,7 +8100,6 @@ const docTemplate = `{ "ExperimentSingleTailnet", "ExperimentTemplateRestartRequirement", "ExperimentDeploymentHealthPage", - "ExperimentTemplateParametersInsights", "ExperimentWorkspacesBatchActions" ] }, diff --git a/coderd/apidoc/swagger.json b/coderd/apidoc/swagger.json index eef5f6d0dae2a..375be4c013fff 100644 --- a/coderd/apidoc/swagger.json +++ b/coderd/apidoc/swagger.json @@ -7246,7 +7246,6 @@ "single_tailnet", "template_restart_requirement", "deployment_health_page", - "template_parameters_insights", "workspaces_batch_actions" ], "x-enum-varnames": [ @@ -7256,7 +7255,6 @@ "ExperimentSingleTailnet", "ExperimentTemplateRestartRequirement", "ExperimentDeploymentHealthPage", - "ExperimentTemplateParametersInsights", "ExperimentWorkspacesBatchActions" ] }, diff --git a/codersdk/deployment.go b/codersdk/deployment.go index d49297dbf8f3f..fca5d89278c21 100644 --- a/codersdk/deployment.go +++ b/codersdk/deployment.go @@ -1928,9 +1928,6 @@ const ( // Deployment health page ExperimentDeploymentHealthPage Experiment = "deployment_health_page" - // Template parameters insights - ExperimentTemplateParametersInsights Experiment = "template_parameters_insights" - // Workspaces batch actions ExperimentWorkspacesBatchActions Experiment = "workspaces_batch_actions" @@ -1944,7 +1941,6 @@ const ( // not be included here and will be essentially hidden. var ExperimentsAll = Experiments{ ExperimentDeploymentHealthPage, - ExperimentTemplateParametersInsights, ExperimentWorkspacesBatchActions, } diff --git a/docs/api/schemas.md b/docs/api/schemas.md index 523a0e0c3c649..8d73e64e84aa6 100644 --- a/docs/api/schemas.md +++ b/docs/api/schemas.md @@ -2710,7 +2710,6 @@ AuthorizationObject can represent a "set" of objects, such as: all workspaces in | `single_tailnet` | | `template_restart_requirement` | | `deployment_health_page` | -| `template_parameters_insights` | | `workspaces_batch_actions` | ## codersdk.Feature diff --git a/site/src/api/typesGenerated.ts b/site/src/api/typesGenerated.ts index 909111cf5e740..ccb37bc923004 100644 --- a/site/src/api/typesGenerated.ts +++ b/site/src/api/typesGenerated.ts @@ -1598,7 +1598,6 @@ export type Experiment = | "moons" | "single_tailnet" | "tailnet_pg_coordinator" - | "template_parameters_insights" | "template_restart_requirement" | "workspace_actions" | "workspaces_batch_actions" @@ -1607,7 +1606,6 @@ export const Experiments: Experiment[] = [ "moons", "single_tailnet", "tailnet_pg_coordinator", - "template_parameters_insights", "template_restart_requirement", "workspace_actions", "workspaces_batch_actions", diff --git a/site/src/pages/DeploySettingsPage/GeneralSettingsPage/GeneralSettingsPageView.stories.tsx b/site/src/pages/DeploySettingsPage/GeneralSettingsPage/GeneralSettingsPageView.stories.tsx index 9e65fca98cc3a..4a11f6224c08a 100644 --- a/site/src/pages/DeploySettingsPage/GeneralSettingsPage/GeneralSettingsPageView.stories.tsx +++ b/site/src/pages/DeploySettingsPage/GeneralSettingsPage/GeneralSettingsPageView.stories.tsx @@ -36,7 +36,6 @@ const meta: Meta = { "workspace_actions", "single_tailnet", "deployment_health_page", - "template_parameters_insights", ], flag_shorthand: "", hidden: false, diff --git a/site/src/pages/TemplatePage/TemplateInsightsPage/TemplateInsightsPage.tsx b/site/src/pages/TemplatePage/TemplateInsightsPage/TemplateInsightsPage.tsx index 584f36828139e..131d13104ded4 100644 --- a/site/src/pages/TemplatePage/TemplateInsightsPage/TemplateInsightsPage.tsx +++ b/site/src/pages/TemplatePage/TemplateInsightsPage/TemplateInsightsPage.tsx @@ -30,12 +30,12 @@ import { subDays, isToday } from "date-fns" import "react-date-range/dist/styles.css" import "react-date-range/dist/theme/default.css" import { DateRange, DateRangeValue } from "./DateRange" -import { useDashboard } from "components/Dashboard/DashboardProvider" import OpenInNewOutlined from "@mui/icons-material/OpenInNewOutlined" import Link from "@mui/material/Link" import CheckCircleOutlined from "@mui/icons-material/CheckCircleOutlined" import CancelOutlined from "@mui/icons-material/CancelOutlined" import { getDateRangeFilter } from "./utils" +import Tooltip from "@mui/material/Tooltip" export default function TemplateInsightsPage() { const now = new Date() @@ -61,10 +61,6 @@ export default function TemplateInsightsPage() { queryKey: ["templates", template.id, "user-latency", insightsFilter], queryFn: () => getInsightsUserLatency(insightsFilter), }) - const dashboard = useDashboard() - const shouldDisplayParameters = - dashboard.experiments.includes("template_parameters_insights") || - process.env.NODE_ENV === "development" return ( <> @@ -77,7 +73,6 @@ export default function TemplateInsightsPage() { } templateInsights={templateInsights} userLatency={userLatency} - shouldDisplayParameters={shouldDisplayParameters} /> ) @@ -86,12 +81,10 @@ export default function TemplateInsightsPage() { export const TemplateInsightsPageView = ({ templateInsights, userLatency, - shouldDisplayParameters, dateRange, }: { templateInsights: TemplateInsightsResponse | undefined userLatency: UserLatencyInsightsResponse | undefined - shouldDisplayParameters: boolean dateRange: ReactNode }) => { return ( @@ -114,12 +107,10 @@ export const TemplateInsightsPageView = ({ sx={{ gridColumn: "span 3" }} data={templateInsights?.report.apps_usage} /> - {shouldDisplayParameters && ( - - )} + ) @@ -349,25 +340,34 @@ const TemplateParametersUsagePanel = ({ + theme.palette.text.secondary, + fontWeight: 500, + fontSize: 13, + cursor: "default", + }} + > + Value + + Count + + {parameter.values .sort((a, b) => b.count - a.count) .map((usage, usageIndex) => ( - {usage.count} - + ))} @@ -378,6 +378,14 @@ const TemplateParametersUsagePanel = ({ ) } +const ParameterUsageRow = styled(Box)(({ theme }) => ({ + display: "flex", + alignItems: "baseline", + justifyContent: "space-between", + padding: theme.spacing(0.5, 0), + gap: theme.spacing(5), +})) + const ParameterUsageLabel = ({ usage, parameter, @@ -386,16 +394,7 @@ const ParameterUsageLabel = ({ parameter: TemplateParameterUsage }) => { if (usage.value.trim() === "") { - return ( - theme.palette.text.secondary, - }} - > - Not set - - ) + return Not set } if (parameter.options) {