Skip to content

Commit

Permalink
Add OneDrive to uppy bundle
Browse files Browse the repository at this point in the history
  • Loading branch information
arturi committed Feb 11, 2020
1 parent 856243a commit f291688
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions packages/uppy/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ exports.StatusBar = require('@uppy/status-bar')
exports.Dropbox = require('@uppy/dropbox')
exports.GoogleDrive = require('@uppy/google-drive')
exports.Instagram = require('@uppy/instagram')
exports.OneDrive = require('@uppy/onedrive')
exports.Url = require('@uppy/url')
exports.Webcam = require('@uppy/webcam')

Expand Down
1 change: 1 addition & 0 deletions packages/uppy/index.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ export { default as StatusBar } from '@uppy/status-bar'
export { default as Dropbox } from '@uppy/dropbox'
export { default as GoogleDrive } from '@uppy/google-drive'
export { default as Instagram } from '@uppy/instagram'
export { default as OneDrive } from '@uppy/onedrive'
export { default as Url } from '@uppy/url'
export { default as Webcam } from '@uppy/webcam'

Expand Down

4 comments on commit f291688

@alexdabast
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi seems that this commit introduced a build error

ERROR in ./node_modules/uppy/index.mjs
Module not found: Error: Can't resolve '@uppy/onedrive' in '/Users/xxx/repo/metronic/node_modules/uppy'

@uppy/onedrive is missing in package.json

@arturi
Copy link
Contributor Author

@arturi arturi commented on f291688 Feb 14, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alexdabast wow, thanks for noticing! Fixing in 3492476

@arturi
Copy link
Contributor Author

@arturi arturi commented on f291688 Feb 15, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be resolved now, published uppy@1.9.2, @alexdabast 👍

@alexdabast
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome thank you ! @arturi

Please sign in to comment.