From 34da849cbd6eae9ea6f0836ac86cd5c96a3eef68 Mon Sep 17 00:00:00 2001 From: Ankita Kinger Date: Tue, 1 Feb 2022 18:06:10 +0530 Subject: [PATCH] avoiding multiple API calls on selecting same role for a user under manage users --- app/client/src/components/ads/TableDropdown.tsx | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/app/client/src/components/ads/TableDropdown.tsx b/app/client/src/components/ads/TableDropdown.tsx index 1360356fbb..3f56d8ef33 100644 --- a/app/client/src/components/ads/TableDropdown.tsx +++ b/app/client/src/components/ads/TableDropdown.tsx @@ -97,9 +97,11 @@ function TableDropdown(props: DropdownProps) { }, [props.selectedIndex]); const optionSelector = (index: number) => { - setSelectedIndex(index); - setSelectedOption(props.options[index]); - props.onSelect && props.onSelect(props.options[index]); + if (index !== selectedIndex) { + setSelectedIndex(index); + setSelectedOption(props.options[index]); + props.onSelect && props.onSelect(props.options[index]); + } setIsDropdownOpen(false); };