fix: add missing deps to registrationResult useEffect
Add USER_RETENTION_COOKIE_NAME and SESSION_COOKIE_DOMAIN to the dependency array instead of suppressing react-hooks/exhaustive-deps. Both are config constants from useAppConfig() that never change at runtime, so behavior is unchanged. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
committed by
Adolfo R. Brandes
parent
86b4ea79de
commit
56d8a10694
@@ -205,7 +205,7 @@ const RegistrationPage = (props) => {
|
||||
// This is used by the "User Retention Rate Event" on GTM
|
||||
setCookie(USER_RETENTION_COOKIE_NAME, true, SESSION_COOKIE_DOMAIN);
|
||||
}
|
||||
}, [registrationResult]); // eslint-disable-line react-hooks/exhaustive-deps
|
||||
}, [registrationResult, USER_RETENTION_COOKIE_NAME, SESSION_COOKIE_DOMAIN]);
|
||||
|
||||
const handleOnChange = (event) => {
|
||||
const { name } = event.target;
|
||||
|
||||
Reference in New Issue
Block a user