Merge pull request #48 from dasuni-30/fix-pr-builder

fix(react): Fix EsLint issues in the PR builder

Author: dasuni-30

.changeset/spicy-lies-study.md

@@ -0,0 +1,5 @@
+---
+'@asgardeo/react': patch
+---
+
+Fix EsLint issues in the PR build failure

packages/react/src/components/SignIn/SignIn.tsx

@@ -82,6 +82,7 @@ const SignIn: FC<SignInProps> = (props: SignInProps): ReactElement => {
   const {config} = useConfig();
 
   const authContext: AuthContext | undefined = useContext(AsgardeoContext);
+  const {setAuthResponse, setIsComponentLoading} = authContext;
   const brandingPreference: Branding = useContext(BrandingPreferenceContext);
 
   const {isLoading, t} = useTranslations({
@@ -102,16 +103,16 @@ const SignIn: FC<SignInProps> = (props: SignInProps): ReactElement => {
      */
     authorize()
       .then((response: AuthApiResponse) => {
-        authContext?.setAuthResponse(response);
+        setAuthResponse(response);
       })
       .catch((error: Error) => {
         setAlert({alertType: {error: true}, key: keys.common.error});
         throw new AsgardeoUIException('REACT_UI-SIGN_IN-SI-SE01', 'Authorization failed', error.stack);
       })
       .finally(() => {
-        authContext?.setIsComponentLoading(false);
+        setIsComponentLoading(false);
       });
-  }, []);
+  }, [setAuthResponse, setIsComponentLoading]);
 
   /**
    * Handles the generalized authentication process.

packages/react/src/hooks/use-on.ts

@@ -38,7 +38,7 @@ const useOn = (props: UseOnProps): void => {
       default:
         throw new AsgardeoUIException('REACT-USE_ON-UO-IV-01', 'Invalid event type provided.');
     }
-  }, []);
+  }, [callback, contextValue, event]);
 };
 
 export default useOn;

packages/react/src/hooks/use-translations.ts

@@ -49,11 +49,11 @@ const useTranslations = (props: SetTranslationsProps): UseTranslations => {
       setIsLoading(!response);
       setIsTextLoading(!response);
     });
-  }, [componentLocaleOverride, componentTextOverrides, screen, setTranslations]);
+  }, [componentLocaleOverride, componentTextOverrides, screen, setIsTextLoading, setTranslations]);
 
   useEffect(() => {
     setIsTextLoading(isLoading);
-  }, [isLoading]);
+  }, [isLoading, setIsTextLoading]);
 
   /**
    * `t` is a function that retrieves a specific translation from the fetched translations.