React 18 StrictMode causes "AuthUI instance is deleted" error
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- firebase, javascript, react
- Área
- authentication, frontend
Línea de trabajo
Comienza localizando las implementaciones de componentDidMount y componentWillUnmount del componente wrapper; después, compara su comportamiento de limpieza con el ciclo de vida de React 18 StrictMode descrito aquí. Reproduce el problema en una aplicación local de React 18 StrictMode y verifica que la limpieza deje utilizable la instancia de AuthUI y que la UI se renderice sin el error de eliminación.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
When using react-firebaseui in React 18 with Strict Mode enabled and running a local dev server, the UI doesn’t render at all and produces this error in the console:
This is because of a change in behavior in Strict Mode to support concurrent features in React 18. Now components are mounted twice. See:
- https://github.com/reactwg/react-18/discussions/19
- https://github.com/reactwg/react-18/discussions/18
It seems the culprit is the deletion of the AuthUI instance when the component is unmounted. Reading the firebaseui documentation, it seems that this isn’t necessary to do, since the firebaseUiWidget property will either get the existing instance or create a new one.
I’ve ported the existing componentDidMount and componentWillUnmount code to a useEffect and removed the instance deletion, and used the modular v9 version of onAuthStateChanged. This seems to work for me:
useEffect(() => {
let firebaseUiWidget: firebaseui.auth.AuthUI;
let userSignedIn = false;
let unregisterAuthObserver: ReturnType<typeof onAuthStateChanged>;
// Get or Create a firebaseUI instance.
firebaseUiWidget =
firebaseui.auth.AuthUI.getInstance() ||
new firebaseui.auth.AuthUI(firebaseAuth);
if (uiConfig.signInFlow === "popup") firebaseUiWidget.reset();
// We track the auth state to reset firebaseUi if the user signs out.
unregisterAuthObserver = onAuthStateChanged(firebaseAuth, (user) => {
if (!user && userSignedIn) firebaseUiWidget.reset();
userSignedIn = !!user;
});
// Render the firebaseUi Widget.
firebaseUiWidget.start("#" + ELEMENT_ID, uiConfig);
return () => {
unregisterAuthObserver();
firebaseUiWidget.reset();
};
}, [uiConfig]);
Is there something that I’m missing that would require the instance to be deleted entirely?
- Lenguaje dominante
- JavaScript
- Estrellas
- 1.3k
- Forks
- 246
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de firebase/firebaseui-web-react
-
This Project Is Dead Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 10/100
firebase/firebaseui-web-react#292 · 1 reacción ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 1/100
firebase/firebaseui-web-react#280 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
firebase/firebaseui-web-react#220 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
firebase/firebaseui-web-react#210 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 20/100
firebase/firebaseui-web-react#201 · 1 comentario · 1 reacción ·
Todos los issues de firebase/firebaseui-web-react
Issues similares
-
Update HugeIcons library Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
antfu-collective/icones#398 ·
-
ECmail.com Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
wesbos/burner-email-providers#554 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
radiantearth/stac-browser#1023 ·
-
HMR stops working Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
components-web-app/docs#92 ·