Browse Source

add useCallback for clickAddLdapAccountHandler

kaori 3 years ago
parent
commit
a9a7951b0c
1 changed files with 3 additions and 3 deletions
  1. 3 3
      packages/app/src/components/Me/AssociateModal.tsx

+ 3 - 3
packages/app/src/components/Me/AssociateModal.tsx

@@ -1,4 +1,4 @@
-import React, { useState } from 'react';
+import React, { useState, useCallback } from 'react';
 
 
 import { useTranslation } from 'react-i18next';
 import { useTranslation } from 'react-i18next';
 import {
 import {
@@ -28,12 +28,12 @@ const AssociateModal = (props: Props): JSX.Element => {
   const [password, setPassword] = useState('');
   const [password, setPassword] = useState('');
 
 
 
 
-  const clickAddLdapAccountHandler = () => {
+  const clickAddLdapAccountHandler = useCallback(() => {
     associateLdapAccount({ username, password });
     associateLdapAccount({ username, password });
     mutatePersonalExternalAccounts();
     mutatePersonalExternalAccounts();
 
 
     onClose();
     onClose();
-  };
+  }, [associateLdapAccount, mutatePersonalExternalAccounts, onClose, password, username]);
 
 
   return (
   return (
     <Modal isOpen={isOpen} toggle={onClose} size="lg" data-testid="grw-associate-modal">
     <Modal isOpen={isOpen} toggle={onClose} size="lg" data-testid="grw-associate-modal">