Skip to content

Commit 0d9197c

Browse files
committed
fix: remove unnecessary useMemo wrapping trivial isDirty check
1 parent 415d6a1 commit 0d9197c

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

apps/webapp/app/components/admin/FeatureFlagsDialog.tsx

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { useFetcher } from "@remix-run/react";
2-
import { useEffect, useMemo, useState } from "react";
2+
import { useEffect, useState } from "react";
33
import {
44
Dialog,
55
DialogContent,
@@ -74,9 +74,7 @@ export function FeatureFlagsDialog({
7474
}
7575
}, [saveFetcher.data]);
7676

77-
const isDirty = useMemo(() => {
78-
return JSON.stringify(overrides) !== JSON.stringify(initialOverrides);
79-
}, [overrides, initialOverrides]);
77+
const isDirty = JSON.stringify(overrides) !== JSON.stringify(initialOverrides);
8078

8179
const setFlagValue = (key: string, value: unknown) => {
8280
setOverrides((prev) => ({ ...prev, [key]: value }));

0 commit comments

Comments
 (0)