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

Upgrade react-element-to-jsx-string dependency of app/react #17076

Closed
wants to merge 2 commits into from
Closed

Upgrade react-element-to-jsx-string dependency of app/react #17076

wants to merge 2 commits into from

Conversation

angry-meow
Copy link

@angry-meow angry-meow commented Dec 24, 2021

Closes #16827 but unfortunately reopens #12747.

react-element-to-jsx-string was updated in #16407 as a bug fix.

But #16827 seems to be a more common problem.

What I did

I edited addons/docs/package.json then ran yarn.

I didn't commit these changes:

  • lib/core-server/src/__snapshots__/cra-ts-essentials_manager-prod-windows
  • lib/core-server/src/__snapshots__/cra-ts-essentials_preview-prod-windows

There were the following addition the the existing prod snapshots:

exports[`cra-ts-essentials manager dev 1`] = `...`;
exports[`cra-ts-essentials preview dev 1`] = `...`;

@nx-cloud
Copy link

nx-cloud bot commented Dec 24, 2021

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 43ef8b7. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

@angry-meow angry-meow changed the title Downgrade react-element-to-jsx-string dependency Downgrade react-element-to-jsx-string dependency of addons/docs Dec 24, 2021
@ndelangen
Copy link
Member

@angry-meow could you investigate "react-element-to-jsx-string" version 15 that seems o have been released 2 months ago? Does this (I hope so) solve both problems?

Thank you for the help!

# Conflicts:
#	addons/docs/package.json
#	yarn.lock
@ndelangen ndelangen changed the title Downgrade react-element-to-jsx-string dependency of addons/docs Upgrade react-element-to-jsx-string dependency of app/react Jun 30, 2022
@ndelangen ndelangen marked this pull request as draft July 6, 2022 18:02
@IanVS
Copy link
Member

IanVS commented Sep 4, 2022

Hi, what's the status of this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Out-of-memory in development mode.
5 participants