Skip to content

Commit 1757a6e

Browse files
author
Joey Marshment-Howell
authored
🪟 🐛 Fix duplicate createUser calls on signup (#21454)
* revert useEffect causing multiple user creations
1 parent c538977 commit 1757a6e

File tree

1 file changed

+1
-18
lines changed

1 file changed

+1
-18
lines changed

airbyte-webapp/src/packages/cloud/services/auth/AuthService.tsx

Lines changed: 1 addition & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { User as FirebaseUser } from "firebase/auth";
2-
import React, { useCallback, useContext, useEffect, useMemo, useRef } from "react";
2+
import React, { useCallback, useContext, useMemo, useRef } from "react";
33
import { useQueryClient } from "react-query";
44
import { useEffectOnce } from "react-use";
55
import { Observable, Subject } from "rxjs";
@@ -160,23 +160,6 @@ export const AuthenticationProvider: React.FC<React.PropsWithChildren<unknown>>
160160
});
161161
});
162162

163-
useEffect(() => {
164-
const onFocus = async () => {
165-
return auth.onAuthStateChanged(async (currentUser) => {
166-
if (!currentUser) {
167-
loggedOut();
168-
} else {
169-
await onAfterAuth(currentUser);
170-
}
171-
});
172-
};
173-
174-
window.addEventListener("focus", onFocus);
175-
return () => {
176-
window.removeEventListener("focus", onFocus);
177-
};
178-
}, [auth, loggedOut, onAfterAuth]);
179-
180163
const queryClient = useQueryClient();
181164

182165
const ctx: AuthContextApi = useMemo(

0 commit comments

Comments
 (0)