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

feat: pass in react #859

Merged
merged 1 commit into from
Apr 12, 2024
Merged

feat: pass in react #859

merged 1 commit into from
Apr 12, 2024

Conversation

kellyjosephprice
Copy link
Collaborator

@kellyjosephprice kellyjosephprice commented Apr 12, 2024

PR App Fix RM-XYZ

🧰 Changes

Allows importing React within mdx.

Aaron pointed out that we couldn't use react features such as useState. The more conventional way to import react would be from a url, but this seemed simpler.

🧬 QA & Testing

@kellyjosephprice kellyjosephprice changed the base branch from next to beta April 12, 2024 00:44
@kellyjosephprice kellyjosephprice marked this pull request as ready for review April 12, 2024 01:05
@kellyjosephprice kellyjosephprice requested review from jennspencer and removed request for erunion April 12, 2024 01:05
@Dashron
Copy link
Contributor

Dashron commented Apr 12, 2024

Sadly the more conventional way of just letting import work, requires the rest of our stack to be using import for react so we get the same one :( more hacks around legacy code.

I do think this is something they should be supporting out of the box though. It doesn't need to be this opinionated.

@kellyjosephprice kellyjosephprice merged commit 34d9bb6 into beta Apr 12, 2024
4 of 5 checks passed
@kellyjosephprice kellyjosephprice deleted the feat/pass-in-react branch April 12, 2024 16:28
rafegoldberg pushed a commit that referenced this pull request Apr 12, 2024
## Version 6.75.0-beta.25

### ✨ New & Improved

* pass in react ([#859](#859)) ([34d9bb6](34d9bb6))

<!--SKIP CI-->
@rafegoldberg
Copy link
Contributor

This PR was released!

🚀 Changes included in v6.75.0-beta.25

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.

3 participants