@uppy/audio: refactor to ESM #3470
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
npm package is still CommonJS, the plan is to switch that in the next semver-major.
Murderlon
reviewed
Mar 9, 2022
| this.#supportsUserMedia = this.#mediaDevices != null | ||
| this.id = this.opts.id || 'Audio' | ||
| this.type = 'acquirer' | ||
| this.icon = () => ( |
There was a problem hiding this comment.
Ideally we would have an SVG loader, because this feels cluttered. But I understand that is out of scope for this PR.
| this.unmount() | ||
| } | ||
| } | ||
| export { default } from './Audio.jsx' |
There was a problem hiding this comment.
Why can't we have this exported file directly in here?
There was a problem hiding this comment.
Because the package entry point has to be lib/index.js (changing that would be a breaking change), and we cannot put JSX in a JS file.
Murderlon
approved these changes
Mar 9, 2022
Murderlon
added a commit
that referenced
this pull request
Mar 14, 2022
* main: Abstract restriction logic in a new Restricter class (#3532) @uppy/companion - Fetch all Google Drive shared drives (#3553) website: add blog post 2.4-2.7 (#3557) meta: fix e2e (#3562) fix broken link (#3559) meta: fix support of export declaration in source files (#3558) Order Google Drive results by folder to show all folders first (#3546) add corsOrigins to docs (#3554) @uppy/audio: refactor to ESM (#3470) @uppy/locales: compressor cleanup (#3531) meta: fix CJS interop in Vite config (#3543) upgrade node-redis-pubsub (#3541)
github-actions bot
added a commit
that referenced
this pull request
Mar 16, 2022
| Package | Version | Package | Version | | -------------------- | ------- | -------------------- | ------- | | @uppy/audio | 0.3.0 | @uppy/locales | 2.0.8 | | @uppy/aws-s3 | 2.0.8 | @uppy/provider-views | 2.0.8 | | @uppy/companion | 3.4.0 | @uppy/vue | 0.4.6 | | @uppy/compressor | 0.2.3 | @uppy/robodog | 2.4.0 | | @uppy/core | 2.1.6 | uppy | 2.8.0 | | @uppy/drop-target | 1.1.2 | | | - @uppy/aws-s3: fix wrong events being sent to companion (Mikael Finstad / #3576) - @uppy/compressor: ignore remote files, calculate savings correctly (Artur Paikin / #3578) - @uppy/companion: always log errors with stack trace (Mikael Finstad / #3573) - meta: remove incorrect s3 documentation (Mikael Finstad / #3571) - @uppy/companion: Companion refactor (Mikael Finstad / #3542) - website: partial ooops (Artur Paikin) - meta: run e2e workflow on the head branch instead of the base one (Antoine du Hamel / #3561) - website: Use Plausible instead of Google Analytics (Artur Paikin / #3567) - @uppy/vue: enforce use of file extension within the import path (Antoine du Hamel / #3560) - @uppy/drop-target: ignore if dropped elements aren't files (Penar Musaraj / #3563) - @uppy/core: Abstract restriction logic in a new Restricter class (Merlijn Vos / #3532) - @uppy/companion: Fetch all Google Drive shared drives (Robert DiMartino / #3553) - website: add blog post 2.4-2.7 (Artur Paikin / #3557) - meta: fix e2e (Antoine du Hamel / #3562) - meta: fix broken link (YukeshShr / #3559) - meta: fix support of export declaration in source files (Antoine du Hamel / #3558) - @uppy/companion: Order Google Drive results by folder to show all folders first (Robert DiMartino / #3546) - meta: add corsOrigins to docs (Mikael Finstad / #3554) - @uppy/audio: refactor to ESM (Antoine du Hamel / #3470) - @uppy/locales: compressor cleanup (Antoine du Hamel / #3531) - meta: fix CJS interop in Vite config (Antoine du Hamel / #3543) - @uppy/companion: upgrade node-redis-pubsub (Mikael Finstad / #3541) - @uppy/provider-views: provider-view: fix breadcrumbs (Artur Paikin / #3535) - meta: Update BACKLOG.md (Artur Paikin) - @uppy/locales: Update ru_RU.js (Sobakin Sviatoslav / #3529) - @uppy/companion: reorder reqToOptions (Antoine du Hamel / #3530) - meta: Fix yarn caching in github actions (Mikael Finstad / #3526)
vymao
pushed a commit
to vymao/uppy
that referenced
this pull request
Mar 29, 2022
* main: Abstract restriction logic in a new Restricter class (transloadit#3532) @uppy/companion - Fetch all Google Drive shared drives (transloadit#3553) website: add blog post 2.4-2.7 (transloadit#3557) meta: fix e2e (transloadit#3562) fix broken link (transloadit#3559) meta: fix support of export declaration in source files (transloadit#3558) Order Google Drive results by folder to show all folders first (transloadit#3546) add corsOrigins to docs (transloadit#3554) @uppy/audio: refactor to ESM (transloadit#3470) @uppy/locales: compressor cleanup (transloadit#3531) meta: fix CJS interop in Vite config (transloadit#3543) upgrade node-redis-pubsub (transloadit#3541)
vymao
pushed a commit
to vymao/uppy
that referenced
this pull request
Mar 29, 2022
| Package | Version | Package | Version | | -------------------- | ------- | -------------------- | ------- | | @uppy/audio | 0.3.0 | @uppy/locales | 2.0.8 | | @uppy/aws-s3 | 2.0.8 | @uppy/provider-views | 2.0.8 | | @uppy/companion | 3.4.0 | @uppy/vue | 0.4.6 | | @uppy/compressor | 0.2.3 | @uppy/robodog | 2.4.0 | | @uppy/core | 2.1.6 | uppy | 2.8.0 | | @uppy/drop-target | 1.1.2 | | | - @uppy/aws-s3: fix wrong events being sent to companion (Mikael Finstad / transloadit#3576) - @uppy/compressor: ignore remote files, calculate savings correctly (Artur Paikin / transloadit#3578) - @uppy/companion: always log errors with stack trace (Mikael Finstad / transloadit#3573) - meta: remove incorrect s3 documentation (Mikael Finstad / transloadit#3571) - @uppy/companion: Companion refactor (Mikael Finstad / transloadit#3542) - website: partial ooops (Artur Paikin) - meta: run e2e workflow on the head branch instead of the base one (Antoine du Hamel / transloadit#3561) - website: Use Plausible instead of Google Analytics (Artur Paikin / transloadit#3567) - @uppy/vue: enforce use of file extension within the import path (Antoine du Hamel / transloadit#3560) - @uppy/drop-target: ignore if dropped elements aren't files (Penar Musaraj / transloadit#3563) - @uppy/core: Abstract restriction logic in a new Restricter class (Merlijn Vos / transloadit#3532) - @uppy/companion: Fetch all Google Drive shared drives (Robert DiMartino / transloadit#3553) - website: add blog post 2.4-2.7 (Artur Paikin / transloadit#3557) - meta: fix e2e (Antoine du Hamel / transloadit#3562) - meta: fix broken link (YukeshShr / transloadit#3559) - meta: fix support of export declaration in source files (Antoine du Hamel / transloadit#3558) - @uppy/companion: Order Google Drive results by folder to show all folders first (Robert DiMartino / transloadit#3546) - meta: add corsOrigins to docs (Mikael Finstad / transloadit#3554) - @uppy/audio: refactor to ESM (Antoine du Hamel / transloadit#3470) - @uppy/locales: compressor cleanup (Antoine du Hamel / transloadit#3531) - meta: fix CJS interop in Vite config (Antoine du Hamel / transloadit#3543) - @uppy/companion: upgrade node-redis-pubsub (Mikael Finstad / transloadit#3541) - @uppy/provider-views: provider-view: fix breadcrumbs (Artur Paikin / transloadit#3535) - meta: Update BACKLOG.md (Artur Paikin) - @uppy/locales: Update ru_RU.js (Sobakin Sviatoslav / transloadit#3529) - @uppy/companion: reorder reqToOptions (Antoine du Hamel / transloadit#3530) - meta: Fix yarn caching in github actions (Mikael Finstad / transloadit#3526)
HeavenFox
pushed a commit
to docsend/uppy
that referenced
this pull request
Jun 27, 2023
| Package | Version | Package | Version | | -------------------- | ------- | -------------------- | ------- | | @uppy/audio | 0.3.0 | @uppy/locales | 2.0.8 | | @uppy/aws-s3 | 2.0.8 | @uppy/provider-views | 2.0.8 | | @uppy/companion | 3.4.0 | @uppy/vue | 0.4.6 | | @uppy/compressor | 0.2.3 | @uppy/robodog | 2.4.0 | | @uppy/core | 2.1.6 | uppy | 2.8.0 | | @uppy/drop-target | 1.1.2 | | | - @uppy/aws-s3: fix wrong events being sent to companion (Mikael Finstad / transloadit#3576) - @uppy/compressor: ignore remote files, calculate savings correctly (Artur Paikin / transloadit#3578) - @uppy/companion: always log errors with stack trace (Mikael Finstad / transloadit#3573) - meta: remove incorrect s3 documentation (Mikael Finstad / transloadit#3571) - @uppy/companion: Companion refactor (Mikael Finstad / transloadit#3542) - website: partial ooops (Artur Paikin) - meta: run e2e workflow on the head branch instead of the base one (Antoine du Hamel / transloadit#3561) - website: Use Plausible instead of Google Analytics (Artur Paikin / transloadit#3567) - @uppy/vue: enforce use of file extension within the import path (Antoine du Hamel / transloadit#3560) - @uppy/drop-target: ignore if dropped elements aren't files (Penar Musaraj / transloadit#3563) - @uppy/core: Abstract restriction logic in a new Restricter class (Merlijn Vos / transloadit#3532) - @uppy/companion: Fetch all Google Drive shared drives (Robert DiMartino / transloadit#3553) - website: add blog post 2.4-2.7 (Artur Paikin / transloadit#3557) - meta: fix e2e (Antoine du Hamel / transloadit#3562) - meta: fix broken link (YukeshShr / transloadit#3559) - meta: fix support of export declaration in source files (Antoine du Hamel / transloadit#3558) - @uppy/companion: Order Google Drive results by folder to show all folders first (Robert DiMartino / transloadit#3546) - meta: add corsOrigins to docs (Mikael Finstad / transloadit#3554) - @uppy/audio: refactor to ESM (Antoine du Hamel / transloadit#3470) - @uppy/locales: compressor cleanup (Antoine du Hamel / transloadit#3531) - meta: fix CJS interop in Vite config (Antoine du Hamel / transloadit#3543) - @uppy/companion: upgrade node-redis-pubsub (Mikael Finstad / transloadit#3541) - @uppy/provider-views: provider-view: fix breadcrumbs (Artur Paikin / transloadit#3535) - meta: Update BACKLOG.md (Artur Paikin) - @uppy/locales: Update ru_RU.js (Sobakin Sviatoslav / transloadit#3529) - @uppy/companion: reorder reqToOptions (Antoine du Hamel / transloadit#3530) - meta: Fix yarn caching in github actions (Mikael Finstad / transloadit#3526)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Blocked on #3468.