Merge pull request #4365 from Infisical/ENG-3491

Add memo to availableConnections to fix infinite re-render issue
This commit is contained in:
x032205
2025-08-12 16:21:41 -07:00
committed by GitHub

View File

@@ -1,4 +1,4 @@
import { useEffect } from "react"; import { useEffect, useMemo } from "react";
import { Controller, useForm } from "react-hook-form"; import { Controller, useForm } from "react-hook-form";
import { SingleValue } from "react-select"; import { SingleValue } from "react-select";
import { zodResolver } from "@hookform/resolvers/zod"; import { zodResolver } from "@hookform/resolvers/zod";
@@ -128,10 +128,10 @@ export const ExternalCaModal = ({ popUp, handlePopUpToggle }: Props) => {
enabled: caType === CaType.ACME enabled: caType === CaType.ACME
}); });
const availableConnections: TAvailableAppConnection[] = [ const availableConnections: TAvailableAppConnection[] = useMemo(
...(availableRoute53Connections || []), () => [...(availableRoute53Connections || []), ...(availableCloudflareConnections || [])],
...(availableCloudflareConnections || []) [availableRoute53Connections, availableCloudflareConnections]
]; );
const isPending = isRoute53Pending || isCloudflarePending; const isPending = isRoute53Pending || isCloudflarePending;