-
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
MIME Type? #2
Comments
Hi! The current code base is just regular vanilla JS with no ES6 syntax, so its not using the regular export syntax making the class currently not importable in es6 projects, (which many react projects are using) So for instance using the line But it might be worth updating the class based on whats happening in the dev world (most stuff will be es6 syntax anyways soon), how are you using react btw, are u using Babel or something else? Do you have a code sample or such so I can try recreating the issue ? |
Hello!
Well that’s very good to know! Thanks so much for informing me.
I am using Babel. There is some code on my github, you can check out MyAstroReport.
… On Apr 16, 2018, at 5:18 AM, Johan Wedfelt ***@***.*** ***@***.***>> wrote:
Hi! The current code base is just regular vanilla JS with no ES6 syntax, so its not using the regular export syntax making the class currently not importable in es6 projects, (which many react projects are using) So for instance using the line import Astrology from 'astrology-js'; will not work. Also the project is not structured for ES6 currently. I might change this if/when I get time, and refactor the class so its also available as an ES6 component. I haven't worked on this project for.. hmm like 3 years 🕺 😄 .. so it sure to be outdated specially when it comes to newer syntax etc.
But it might be worth updating the class based on whats happening in the dev world (most stuff will be es6 syntax anyways soon), how are you using react btw, are u using Babel or something else? Do you have a code sample or such so I can try recreating the issue ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#2 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/Af7SXh3X-psT3xFo3IATYErB-tq7mMtLks5tpIwhgaJpZM4TGE0m>.
|
I am unable to get this class to work in my code (using React).
Getting an error that the MIME type is text/HTML.
It is also not able to identify what Astrology() is.
I did the npm install and included the script.
The text was updated successfully, but these errors were encountered: