quick fix (#4154)

web/src/components/admin/connectors/AccessTypeForm.tsx

@@ -29,20 +29,24 @@ export function AccessTypeForm({
   const isAutoSyncSupported = isValidAutoSyncSource(connector);
   const { isAdmin } = useUser();
 
-  useEffect(() => {
+  useEffect(
-    if (!isPaidEnterpriseEnabled) {
+    () => {
-      access_type_helpers.setValue("public");
+      // Only set default value if access_type.value is not already set
-    } else if (isAutoSyncSupported) {
+      if (!access_type.value) {
-      access_type_helpers.setValue("sync");
+        if (!isPaidEnterpriseEnabled) {
-    } else {
+          access_type_helpers.setValue("public");
-      access_type_helpers.setValue("private");
+        } else if (isAutoSyncSupported) {
-    }
+          access_type_helpers.setValue("sync");
-  }, [
+        } else {
-    isAutoSyncSupported,
+          access_type_helpers.setValue("private");
-    isAdmin,
+        }
-    isPaidEnterpriseEnabled,
+      }
-    access_type_helpers,
+    },
-  ]);
+    [
+      // Only run this effect once when the component mounts
+      // eslint-disable-next-line react-hooks/exhaustive-deps
+    ]
+  );
 
   const options = [
     {
@@ -81,7 +85,6 @@ export function AccessTypeForm({
               Control who has access to the documents indexed by this connector.
             </p>
           </div>
-
           <DefaultDropdown
             options={options}
             selected={access_type.value}
@@ -90,7 +93,6 @@ export function AccessTypeForm({
             }
             includeDefault={false}
           />
-
           {access_type.value === "sync" && isAutoSyncSupported && (
             <AutoSyncOptions connectorType={connector as ValidAutoSyncSource} />
           )}