-
Notifications
You must be signed in to change notification settings - Fork 22
netlify-cms-app
import failed to resolve with astro's react integration
#51
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
Comments
Here's a link to the repo: astro-netlifycms-react And a trace of the failed netlify deploy:
|
After looking through some source code I found this My tsx components seem to be working perfectly after leaving it out, rebuilding and serving. Would be helpful to have this piece of information in the README imo |
I'm getting the exact same error message using astro-netlify-cms version 0.5.0 with the react integration. This is the trace:
|
Thanks @danxschz! Any chance you can share your project or a minimal reproduction, e.g. on StackBlitz? |
|
Perfect, thank you. I’m very curious what’s going on. Made my own minimal set-up that didn’t do this — https://stackblitz.com/edit/github-u7myzp?file=astro.config.mjs — so it’s helpful to have your example to test with. |
OK, so one quick fix: downgrading to React 17 and installing npm i {,@types/}{react,react-dom}@^17 netlify-cms-app Don’t understand why this would be necessary in one project and not another but will keep investigating. |
Thank you @delucis, I had the same problem and now it's ok 👌 |
For anyone using pnpm and having the same problem, remember to
|
Running into the following error:
Seems to be related to #34 , but with
react()
instead ofvue()
added as integration.Makes no difference if putting it before or after
NetlifyCMS()
in the config.The admin dashboard is working perfectly fine without the react integration and dependencies.
Will provide a cleaned repo to reproduce the issue later.
The text was updated successfully, but these errors were encountered: