fix: clarify comments on optimistic updates and server state handling in Security component

2026-03-18 22:06:40 +00:00
parent 9a07619b89
commit 5505f66c41
1 changed files with 3 additions and 1 deletions
--- a/frontend/src/pages/Security.tsx
+++ b/frontend/src/pages/Security.tsx
@@ -197,7 +197,9 @@ export default function Security() {
        return { enabled: false }
      }
    },
-    // NO optimistic updates - wait for actual confirmation
+    // No optimistic backend/status invalidation — server state is not updated until
+    // onSuccess. The UI does derive checked state from mutation variables while
+    // isPending to reflect the user's intent immediately (see crowdsecChecked).
    onMutate: async (enabled: boolean) => {
      if (enabled) {
        queryClient.setQueryData(['crowdsec-starting'], { isStarting: true, startedAt: Date.now() })