Przeglądaj źródła

Merge branch 'main' of https://github.com/transloadit/uppy

Antoine du Hamel 1 rok temu
rodzic
commit
7681b7a221

+ 1 - 1
.eslintrc.js

@@ -471,7 +471,7 @@ module.exports = {
       files: ['packages/@uppy/*/src/**/*.ts', 'packages/@uppy/*/src/**/*.tsx'],
       excludedFiles: ['packages/@uppy/**/*.test.ts', 'packages/@uppy/core/src/mocks/*.ts'],
       rules: {
-        '@typescript-eslint/explicit-function-return-type': 'error',
+        '@typescript-eslint/explicit-module-boundary-types': 'error',
       },
     },
     {

+ 2 - 2
packages/@uppy/core/src/BasePlugin.ts

@@ -13,7 +13,7 @@
 import Translator from '@uppy/utils/lib/Translator'
 import type { I18n, Locale } from '@uppy/utils/lib/Translator'
 import type { Body, Meta } from '@uppy/utils/lib/UppyFile'
-import type { Uppy } from '.'
+import type { State, Uppy } from './Uppy'
 
 export type PluginOpts = {
   locale?: Locale
@@ -103,7 +103,7 @@ export default class BasePlugin<
   uninstall(): void {}
 
   // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  update(state: any): void {}
+  update(state: Partial<State<M, B>>): void {}
 
   // Called after every state update, after everything's mounted. Debounced.
   afterUpdate(): void {}

+ 3 - 2
packages/@uppy/core/src/UIPlugin.ts

@@ -7,6 +7,7 @@ import getTextDirection from '@uppy/utils/lib/getTextDirection'
 import type { Body, Meta } from '@uppy/utils/lib/UppyFile'
 import BasePlugin from './BasePlugin.ts'
 import type { PluginOpts } from './BasePlugin.ts'
+import type { State } from './Uppy.ts'
 
 /**
  * Defer a frequent call to the microtask queue.
@@ -48,7 +49,7 @@ class UIPlugin<
   M extends Meta,
   B extends Body,
 > extends BasePlugin<Opts, M, B> {
-  #updateUI: (state: any) => void
+  #updateUI: (state: Partial<State<M, B>>) => void
 
   isTargetDOMEl: boolean
 
@@ -175,7 +176,7 @@ class UIPlugin<
     )
   }
 
-  update(state: any): void {
+  update(state: Partial<State<M, B>>): void {
     if (this.el != null) {
       this.#updateUI?.(state)
     }

+ 4 - 1
packages/@uppy/form/src/index.js

@@ -12,6 +12,9 @@ import packageJson from '../package.json'
 export default class Form extends BasePlugin {
   static VERSION = packageJson.version
 
+  /** @type {HTMLFormElement} */
+  form // TODO: make this private (or at least, mark it as readonly)
+
   constructor (uppy, opts) {
     super(uppy, opts)
     this.type = 'acquirer'
@@ -50,7 +53,7 @@ export default class Form extends BasePlugin {
     }
 
     if (this.opts.submitOnSuccess) {
-      this.form.submit()
+      this.form.requestSubmit()
     }
   }
 

+ 1 - 1
packages/@uppy/provider-views/src/Item/components/ItemIcon.jsx

@@ -38,7 +38,7 @@ export default (props) => {
       return <VideoIcon />
     default: {
       const { alt } = props
-      return <img src={itemIconString} alt={alt} loading="lazy" width={16} height={16} />
+      return <img src={itemIconString} alt={alt} referrerPolicy="no-referrer" loading="lazy" width={16} height={16} />
     }
   }
 }