Skip to content

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

Merged
merged 2 commits into from
May 6, 2025
Merged

Enable React 19 usage (#7710) #7735

merged 2 commits into from
May 6, 2025

Conversation

tnorling
Copy link
Collaborator

@tnorling tnorling commented May 6, 2025

PR to run CI against #7710 and get it merged into dev. Thanks @rm-eth-a-n for making the changes

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 <[email protected]>
@github-actions github-actions bot added documentation Related to documentation. samples Related to the samples apps for the library. msal-react Related to @azure/msal-react labels May 6, 2025
@h3rmanj
Copy link
Contributor

h3rmanj commented May 6, 2025

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

@tnorling
Copy link
Collaborator Author

tnorling commented May 6, 2025

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

sameerag
sameerag previously approved these changes May 6, 2025
@tnorling
Copy link
Collaborator Author

tnorling commented May 6, 2025

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.

@tnorling tnorling merged commit b609846 into dev May 6, 2025
7 checks passed
@tnorling tnorling deleted the react-19 branch May 6, 2025 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Related to documentation. msal-react Related to @azure/msal-react samples Related to the samples apps for the library.
Projects
None yet
6 participants