-
Notifications
You must be signed in to change notification settings - Fork 13.5k
feat(react): fixing support for react 19, adding test app for react 19 #30217
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
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are extra files that are not related to React 19. It seems that they're coming from the fact that feature-8.5
is behind on main
. I recommend syncing feature-8.5
to make sure this PR only has the files related to 19.
Other than that, I have one other comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Issue number: resolves #29991
What is the current behavior?
Currently, there is no test app for React 19. Also, the output test in the react 18 test app is broken because it uses vitest but that file is trying to use jest still.
Additionally, there's issues with using Ionic Framework with React 19 due to several typescript changes. E.g. Issue #29991
What is the new behavior?
Now, there is a test app for React 19 and React 19 should work with Ionic Framework
Does this introduce a breaking change?
Other information