-
Notifications
You must be signed in to change notification settings - Fork 186
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
React 18 support #261
Comments
Yep, started on this a few weeks ago but had to park for a bit. Was playing around with the approach, trying to avoid config if possible. I'll loop back. |
@michaeltaranto Greate news! Thank you! |
any news on this? React is no longer required to be imported into components, and in fact recent typescript linting changes mean that you will get an error when you are importing react. However this then breaks playroom. |
Picked this up again between tasks and thought i'd share an update:
Really want to get this work through, but our effort is definitely focused elsewhere in our ecosystem due to company priorities. I'll try to keep this issue updated. |
Actually, pivoted on this one. Given React still lets you use legacy lifecycle APIs, we have taken in a copy of the editor as a temporary workaround, while we look at upgrading to CodeMirror 6 separately. This allows us to support React 18 (with backwards compatibility and no config/api change). |
Playroom does not support React 18 dependencies. It would be greate to add this support.
The text was updated successfully, but these errors were encountered: