-
Notifications
You must be signed in to change notification settings - Fork 150
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
Does it work on React 14 or 15? #47
Comments
I'm getting the same error. I'm using: |
Could I help to get this working on newer versions of React? Do you accept pull requests? |
@mrn3 a PR would be great, thanks! |
We use this library successfully with the following stack:
Make sure your
|
great thanks! |
Yeah it's unfortunate and unconventional that the module can't be required without transpiling jsx. By default, for example, babel-register ignores |
For webpack2, this load configuration worked for me:
With the packages
|
has anyone been able to get this to work in the default create-react-app configuration? I am not sure if it's possible to modify the loading configurations without ejecting, which I don't want to do yet.. |
If anyone can get this to work, a PR would be great. Thanks @ainsleys. |
I don't mind having a stab at making this work. However the only way I can think of doing it is to add webpack. The reason for this is that the high-level strategy is to convert the |
So the idea is take webpack as a dev dependency and to do all the transpilation before publish. I think that could work. Curious if this will mess with the default styles option. |
I had the same problem, tried all the solutions, nothing worked. I guess I will have to use another library for now. @davidguttman please let us know if you get a fix for this. Browserify is kinda old and will be hard to use it just for you library. |
I'm also having problems setting up this module with react-create-app. As @davidguttman asks for a PR I think there will not be a solution very fast :( I'm getting a
I like the demo of the module but unfortunately can't rewrite it myself to work with react-create-app (and create the asked PR). So does anyone know a similar react module which works with react-create-app? @ainsleys How did you solve it? What are you using now? |
I got this error: InvalidCharacterError: String contains an invalid character |
Is there any update on this issue? |
I am on React 15 and getting:
35% 70/163 build modulesModuleParseError: Module parse failed: /app/reports/node_modules/react-pivot/index.jsx Line 113: Unexpected token <
Has this been updated to work on React 0.14 or 15?
The text was updated successfully, but these errors were encountered: