-
Notifications
You must be signed in to change notification settings - Fork 21
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
the latest published ove package can not be imported correctly. #13
Comments
Hi @sunm19810-pki , Version 0.3.7 seems to be working in this simple react demo page - https://github.com/tnrich/ove-react-demo-repo Does that demo work for you? In what ways is your set up different from that one? Thanks |
@tnrich, it seems works fine with
|
@tnrich you can try this repo |
Hi @tnrich, if we want to make the latest ove package been easily used in webpack, we may need to change some configurations in the |
@sunm19810-pki thanks for looking into this. I'd be happy to review a PR if you make one. |
Ok, @XingGao-PKI @sunm19810-pki I published a new version of ove 0.3.9 that should work for vite and webpack builds. I added a vite and webpack demo to tg-oss To try out the webpack demo within tg-oss run the following commands:
You should see something like this: Lemme know if that works for y'all |
Thanks very much @tnrich, it indeed work now. We actually had similar code PR for webpack building, but we decide to use the previous version as the new version may need some time, it is really good as the new version is published so fast. |
Awesome, glad to hear it! |
Hi @tnrich, it seems the latest 'ove' version (0.3.7) can not be imported correctly, can we publish the 'ove' package and other packages like 'ui' without 'react' or 'react-dom' bundled (open-vector-editor do not set these two packages in the dependencies, it set the two in the peerDependencies and devDependencies)
currently published npm packages can not be used as es module easily. Thanks
The text was updated successfully, but these errors were encountered: