Skip to content

[WIP] Support React 18 - #487

Closed
jhuleatt wants to merge 11 commits into
mainfrom
react-18
Closed

[WIP] Support React 18#487
jhuleatt wants to merge 11 commits into
mainfrom
react-18

Conversation

@jhuleatt

@jhuleatt jhuleatt commented Nov 17, 2021

Copy link
Copy Markdown
Collaborator

React 18 is now in beta.

This PR will investigate how we can/should use the new APIs available in React 18:

useSyncExternalStore

This can probably replace the bulk of the useEffect/useReducer stuff in useObservable 🎉

useId

Maybe this can help with caching?

useInsertionEffect

From the working group discussion:

The use case is specifically for inserting global DOM nodes like <style> or SVG . (Maybe global Portals.) It's not really meant to be used by anything else other than these CSS libraries.

Maybe useful for react-firebaseui?

@google-cla google-cla Bot added the cla: yes label Nov 17, 2021
@jhuleatt
jhuleatt marked this pull request as ready for review May 17, 2022 13:04
@jhuleatt

Copy link
Copy Markdown
Collaborator Author

Note: BehaviorSubject now has a synchronous getValue method: https://rxjs.dev/api/index/class/BehaviorSubject#getvalue

@jhuleatt

Copy link
Copy Markdown
Collaborator Author

implemented in #583

@jhuleatt jhuleatt closed this Jul 24, 2023
@FirebaseExtended FirebaseExtended locked and limited conversation to collaborators Aug 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant