Skip to content
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

JP/single-spa-react #600

Closed
wants to merge 3 commits into from
Closed

JP/single-spa-react #600

wants to merge 3 commits into from

Conversation

webJose
Copy link
Contributor

@webJose webJose commented Oct 23, 2023

Update example for React versions lower than v18.

@webJose
Copy link
Contributor Author

webJose commented Oct 23, 2023

Here are some screenshots of the changes. Apologies, I was test driving the GitHub PR extension in VS Code. It works nicely, but I could not add these shots in the original message.

image

image

@joeldenning
Copy link
Member

I think we should fix latest single-spa-react to work with React 17, rather than update the documentation for it. See single-spa/single-spa-react#202

@joeldenning
Copy link
Member

This documentation change is no longer necessary, due to the fix in [email protected]

https://github.com/single-spa/single-spa-react/releases/tag/v6.0.1

@joeldenning joeldenning closed this Jan 5, 2024
@webJose webJose deleted the JP/single-spa-react branch January 7, 2024 04:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants