From 4774a9ce20e8b76ca76cc0c8a41db33762a6d9f0 Mon Sep 17 00:00:00 2001 From: Usman Baig Date: Thu, 5 Feb 2026 14:39:35 +0100 Subject: [PATCH] refactor: simplify state updates in OrganizationSettings by removing unnecessary useEffect for filters --- components/settings/OrganizationSettings.tsx | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/components/settings/OrganizationSettings.tsx b/components/settings/OrganizationSettings.tsx index fb4f6c7..a140c24 100644 --- a/components/settings/OrganizationSettings.tsx +++ b/components/settings/OrganizationSettings.tsx @@ -99,15 +99,13 @@ export default function OrganizationSettings() { endDate: auditEndDate }) - // Update refs when state changes - useEffect(() => { - filtersRef.current = { - action: auditActionFilter, - logId: auditLogIdFilter, - startDate: auditStartDate, - endDate: auditEndDate - } - }, [auditActionFilter, auditLogIdFilter, auditStartDate, auditEndDate]) + // Update refs when state changes (no useEffect needed) + filtersRef.current = { + action: auditActionFilter, + logId: auditLogIdFilter, + startDate: auditStartDate, + endDate: auditEndDate + } const getOrgIdFromToken = () => { return user?.org_id || null @@ -881,6 +879,7 @@ export default function OrganizationSettings() { setAuditStartDate('') setAuditEndDate('') setAuditPage(0) + setAuditFetchTrigger(prev => prev + 1) }} disabled={isLoadingAudit} >