Skip to content

Commit 5e0cb37

Browse files
fix(site): fix orphan values on insights (#10036)
1 parent 2405bbe commit 5e0cb37

File tree

2 files changed

+16
-0
lines changed

2 files changed

+16
-0
lines changed

site/src/pages/TemplatePage/TemplateInsightsPage/TemplateInsightsPage.stories.tsx

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -162,6 +162,11 @@ export const Loaded: Story = {
162162
value: "us-west2-a",
163163
count: 1,
164164
},
165+
// Test orphan values
166+
{
167+
value: "us-west-orphan",
168+
count: 1,
169+
},
165170
],
166171
},
167172
{

site/src/pages/TemplatePage/TemplateInsightsPage/TemplateInsightsPage.tsx

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -436,6 +436,7 @@ const TemplateParametersUsagePanel = ({
436436
</ParameterUsageRow>
437437
{parameter.values
438438
.sort((a, b) => b.count - a.count)
439+
.filter((usage) => filterOrphanValues(usage, parameter))
439440
.map((usage, usageIndex) => (
440441
<ParameterUsageRow
441442
key={`${parameterIndex}-${usageIndex}`}
@@ -456,6 +457,16 @@ const TemplateParametersUsagePanel = ({
456457
);
457458
};
458459

460+
const filterOrphanValues = (
461+
usage: TemplateParameterValue,
462+
parameter: TemplateParameterUsage,
463+
) => {
464+
if (parameter.options) {
465+
return parameter.options.some((o) => o.value === usage.value);
466+
}
467+
return true;
468+
};
469+
459470
const ParameterUsageRow = styled(Box)(({ theme }) => ({
460471
display: "flex",
461472
alignItems: "baseline",

0 commit comments

Comments
 (0)