-
Notifications
You must be signed in to change notification settings - Fork 352
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
import with full path #258
Comments
Thanks! I'll send this along to the team. |
can gladly make a PR if you like me too |
We welcome PRs if you wish to contribute them, but I can't promise any particular timeline for reviewing and merging them. |
Hello, Jimmy, we are publishing a new major version of the SDK which will hopefully update a good amount of the legacy logic. If this issue continues to persist beyond this update, please feel free to make a PR and I can review it. |
@jimmywarting We've updated this in the SDK, so you should now be able to import it like this:
Hope this helps! |
thx |
if you are going to allow es6 import/export
than you should really require extensions on your path. node handle it just fine but browsers dose not.
i can't simply just do this in the browser without having to bundle it first
it tries to load
https://cdn.jsdelivr.net/npm/dropbox@4.0.30/es/team/dropbox-team
when it should be loadinghttps://cdn.jsdelivr.net/npm/dropbox@4.0.30/es/team/dropbox-team.js
Ryan Dahl regrets the hole package.js and modules resolver.
this is not how browser dose things.
https://youtu.be/M3BM9TB-8yA?t=584
The text was updated successfully, but these errors were encountered: