Skip to content

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

Merged
merged 11 commits into from
Mar 3, 2025

Conversation

ShaneK
Copy link
Member

@ShaneK ShaneK commented Feb 26, 2025

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

  • Additionally, fixed the output test for react 18 by moving to vitest
  • Going forward, react test apps will default to the vitest test app

Does this introduce a breaking change?

  • Yes
  • No

Other information

@ShaneK ShaneK requested a review from a team as a code owner February 26, 2025 19:10
@ShaneK ShaneK requested a review from gnbm February 26, 2025 19:10
Copy link

vercel bot commented Feb 26, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ionic-framework ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 3, 2025 2:51pm

@github-actions github-actions bot added the package: react @ionic/react package label Feb 26, 2025
@ShaneK ShaneK changed the title feat(react): Adding test app for React 19 feat(react): adding test app for React 19 Feb 26, 2025
@ShaneK ShaneK changed the title feat(react): adding test app for React 19 feat(react): adding test app for react 19 Feb 26, 2025
@ShaneK ShaneK changed the title feat(react): adding test app for react 19 feat(react): adding test app for react 19, fixing framework support for react 19 Feb 26, 2025
@ShaneK ShaneK changed the title feat(react): adding test app for react 19, fixing framework support for react 19 feat(react): fixing support for react 19, adding test app for react 19 Feb 27, 2025
@ShaneK ShaneK changed the base branch from main to feature-8.5 February 28, 2025 15:55
@github-actions github-actions bot added the package: core @ionic/core package label Feb 28, 2025
Copy link
Contributor

@thetaPC thetaPC left a 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.

Copy link
Contributor

@thetaPC thetaPC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ShaneK ShaneK merged commit f4941f2 into feature-8.5 Mar 3, 2025
48 checks passed
@ShaneK ShaneK deleted the ROU-11393 branch March 3, 2025 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: react @ionic/react package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: Issues with React 19 types
3 participants