-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Enable React 19 usage (#7710) #7735
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Enable React 19 applications to use this package, it was released on a while back now: https://react.dev/blog/2024/12/05/react-19. This PR enables applications using react 19 to consume the package, while continuing to allow react 18 applications to continue using this package unhindered. --------- Co-authored-by: Herman Jensen <herman.jensen@intility.no>
There's one fix in the Electron node sample needed, as the JSX namespace was moved to React.JSX in react 19 rm-eth-a-n#3 |
Let's wait for the results of CI and any needed changes can be PR'd into this branch |
Looks like the failures are all related to our internal repo which is not open source. I'll make the needed changes later today or tomorrow and get this merged in. Thanks again! @h3rmanj If you want to PR your change into this branch feel free. I can otherwise take care of it later given it's just a one liner. |
PR to run CI against #7710 and get it merged into dev. Thanks @rm-eth-a-n for making the changes --------- Co-authored-by: rm-eth-a-n <ethan.pearce@rightmove.co.uk> Co-authored-by: Herman Jensen <herman.jensen@intility.no>
PR to run CI against #7710 and get it merged into dev. Thanks @rm-eth-a-n for making the changes