@uppy/transloadit: remove deprecated options

[Murderlon]

In preparation for #4765

[github-actions]

Diff output files

diff --git a/packages/@uppy/transloadit/lib/index.js b/packages/@uppy/transloadit/lib/index.js
index 82409b5..928b179 100644
--- a/packages/@uppy/transloadit/lib/index.js
+++ b/packages/@uppy/transloadit/lib/index.js
@@ -14,7 +14,7 @@ import ErrorWithCause from "@uppy/utils/lib/ErrorWithCause";
 import hasProperty from "@uppy/utils/lib/hasProperty";
 import { RateLimitedQueue } from "@uppy/utils/lib/RateLimitedQueue";
 import Assembly from "./Assembly.js";
-import AssemblyOptionsBuilder, { validateParams } from "./AssemblyOptions.js";
+import AssemblyOptionsBuilder from "./AssemblyOptions.js";
 import AssemblyWatcher from "./AssemblyWatcher.js";
 import Client from "./Client.js";
 import locale from "./locale.js";
@@ -37,10 +37,6 @@ const defaultOptions = {
   waitForMetadata: false,
   alwaysRunAssembly: false,
   importFromUploadURLs: false,
-  signature: null,
-  params: null,
-  fields: null,
-  getAssemblyOptions: null,
   limit: 20,
   retryDelays: [7000, 10000, 15000, 20000],
   clientName: null,
@@ -69,7 +65,7 @@ var _onError = _classPrivateFieldLooseKey("onError");
 var _onTusError = _classPrivateFieldLooseKey("onTusError");
 export default class Transloadit extends BasePlugin {
   constructor(uppy, opts) {
-    var _this, _this$opts, _this$opts$assemblyOp, _this$opts$getAssembl;
+    var _this;
     super(uppy, {
       ...defaultOptions,
       ...opts,
@@ -300,8 +296,8 @@ export default class Transloadit extends BasePlugin {
             [uploadID]: [],
           },
         });
-        const assemblyOptions = new AssemblyOptionsBuilder(filesWithoutErrors, this.opts);
-        await assemblyOptions.build().then(assemblies => Promise.all(assemblies.map(createAssembly))).then(
+        const builder = new AssemblyOptionsBuilder(filesWithoutErrors, this.opts);
+        await builder.build().then(assemblies => Promise.all(assemblies.map(createAssembly))).then(
           maybeCreatedAssemblies => {
             const createdAssemblies = maybeCreatedAssemblies.filter(Boolean);
             const assemblyIDs = createdAssemblies.map(assembly => assembly.status.assembly_id);
@@ -414,20 +410,6 @@ export default class Transloadit extends BasePlugin {
     this.type = "uploader";
     this.id = this.opts.id || "Transloadit";
     this.defaultLocale = locale;
-    (_this$opts$assemblyOp = (_this$opts = this.opts).assemblyOptions) != null
-      ? _this$opts$assemblyOp
-      : _this$opts.assemblyOptions = (_this$opts$getAssembl = this.opts.getAssemblyOptions) != null
-        ? _this$opts$getAssembl
-        : {
-          params: this.opts.params,
-          signature: this.opts.signature,
-          fields: this.opts.fields,
-        };
-    if (
-      (opts == null ? void 0 : opts.params) != null && opts.getAssemblyOptions == null && opts.assemblyOptions == null
-    ) {
-      validateParams(this.opts.assemblyOptions.params);
-    }
     _classPrivateFieldLooseBase(this, _rateLimitedQueue)[_rateLimitedQueue] = new RateLimitedQueue(this.opts.limit);
     this.i18nInit();
     this.client = new Client({
@@ -836,6 +818,4 @@ function _connectAssembly2(assembly) {
   return assembly;
 }
 Transloadit.VERSION = packageJson.version;
-Transloadit.COMPANION = COMPANION_URL;
-Transloadit.COMPANION_PATTERN = COMPANION_ALLOWED_HOSTS;
 export { COMPANION_ALLOWED_HOSTS, COMPANION_URL };

[aduh95]

There are some CI failures to address

[kvz]

While we are breaking, can we rename service to endpoint as to be consistent with other integrations?

[aduh95]

We should probably support both, and keep service as a deprecated alias

[Murderlon]

I can do it in a separate PR 👍