DEV: Add addComposerUploadPreProcessor to plugin-api (PR #14222)

This new interface will be used explicitly to add upload preprocessors in the form of uppy plugins. These will be run for each upload in the composer (dependent on the logic of the plugin itself), before the UppyChecksum plugin is finally run.

Since discourse-encrypt uses the existing addComposerUploadHandler API for essentially preprocessing an upload and not uploading it to a different place, it will be the first plugin to use this interface, along with the register-media-optimization-upload-processor initializer in core.

GitHub

This pull request introduces 1 alert when merging 8261f1e33de76e56abcf5c9d143bb691f9a331d9 into fa66d1fa82d6c768f265d3c3601110dca672a205 - view on LGTM.com

new alerts:

  • 1 for Implicit operand conversion

This pull request introduces 1 alert when merging df4c902a98b76c342ce475f250f132dbaac5b34c into fa66d1fa82d6c768f265d3c3601110dca672a205 - view on LGTM.com

new alerts:

  • 1 for Implicit operand conversion

Looks good but we should make sure the clean up is called!

Shouldn’t this be called from tests?

Ah knew I forgot something :slight_smile: Will fix