Skip to content

Commit a604f17

Browse files
committed
Review comments
Signed-off-by: Danny Kopping <danny@coder.com>
1 parent 8836f25 commit a604f17

File tree

1 file changed

+5
-16
lines changed

1 file changed

+5
-16
lines changed

site/src/pages/DeploySettingsPage/GeneralSettingsPage/GeneralSettingsPage.tsx

Lines changed: 5 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -15,22 +15,11 @@ const GeneralSettingsPage: FC = () => {
1515
const enabledExperimentsQuery = useQuery(experiments());
1616
const safeExperimentsQuery = useQuery(availableExperiments());
1717

18-
const safeExperiments: string[] = [];
19-
const invalidExperiments: string[] = [];
20-
21-
(safeExperimentsQuery.data?.safe ?? []).forEach((value) =>
22-
safeExperiments.push(value),
23-
);
24-
25-
(enabledExperimentsQuery.data ?? []).forEach(function (exp) {
26-
const found = (safeExperimentsQuery.data?.safe ?? []).find((value) => {
27-
return exp === value;
28-
});
29-
30-
if (!found) {
31-
invalidExperiments.push(exp);
32-
}
33-
});
18+
const safeExperiments = safeExperimentsQuery.data?.safe ?? [];
19+
const invalidExperiments =
20+
enabledExperimentsQuery.data?.filter(exp => {
21+
return !safeExperiments.includes(exp);
22+
}) ?? [];
3423

3524
return (
3625
<>

0 commit comments

Comments
 (0)