Improve types of variable plugin config

lblod · Apr 17, 2024 · 10a641c · 10a641c
1 parent 2b3babf
commit 10a641c
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 19 deletions.
diff --git a/addon/components/variable-plugin/insert-variable-card.ts b/addon/components/variable-plugin/insert-variable-card.ts
@@ -7,9 +7,16 @@ import { hasGroups } from '@lblod/ember-rdfa-editor/utils/node-utils';
 import { service } from '@ember/service';
 import IntlService from 'ember-intl/services/intl';
 
+type VariableComponentArgs = {
+  Args: {
+    Named: {
+      controller: SayController;
+    };
+  };
+};
 export type VariableConfig = {
   label: string;
-  component: ComponentLike;
+  component: ComponentLike<VariableComponentArgs>;
   options?: unknown;
 };
 

diff --git a/tests/dummy/app/controllers/editable-node.ts b/tests/dummy/app/controllers/editable-node.ts
@@ -204,34 +204,28 @@ export default class EditableBlockController extends Controller {
     return [
       {
         label: 'text',
-        // @ts-expect-error Unclear why these types don't agree
         component: TextVariableInsertComponent,
       },
       {
         label: 'number',
-        // @ts-expect-error Unclear why these types don't agree
         component: NumberInsertComponent,
       },
       {
         label: 'date',
-        // @ts-expect-error Unclear why these types don't agree
         component: DateInsertVariableComponent,
       },
       {
         label: 'location',
-        // @ts-expect-error Unclear why these types don't agree
         component: LocationInsertComponent,
         options: this.locationOptions,
       },
       {
         label: 'codelist',
-        // @ts-expect-error Unclear why these types don't agree
         component: CodelistInsertComponent,
         options: this.codelistOptions,
       },
       {
         label: 'address',
-        // @ts-expect-error Unclear why these types don't agree
         component: VariablePluginAddressInsertVariableComponent,
       },
     ];

diff --git a/tests/dummy/app/controllers/editable-nodes-regulatory-statement.ts b/tests/dummy/app/controllers/editable-nodes-regulatory-statement.ts
@@ -220,34 +220,28 @@ export default class RegulatoryStatementSampleController extends Controller {
     return [
       {
         label: 'text',
-        // @ts-expect-error Unclear why these types don't agree
         component: TextVariableInsertComponent,
       },
       {
         label: 'number',
-        // @ts-expect-error Unclear why these types don't agree
         component: NumberInsertComponent,
       },
       {
         label: 'date',
-        // @ts-expect-error Unclear why these types don't agree
         component: DateInsertVariableComponent,
       },
       {
         label: 'location',
-        // @ts-expect-error Unclear why these types don't agree
         component: LocationInsertComponent,
         options: this.locationOptions,
       },
       {
         label: 'codelist',
-        // @ts-expect-error Unclear why these types don't agree
         component: CodelistInsertComponent,
         options: this.codelistOptions,
       },
       {
         label: 'address',
-        // @ts-expect-error Unclear why these types don't agree
         component: VariablePluginAddressInsertVariableComponent,
       },
     ];

diff --git a/tests/dummy/app/controllers/regulatory-statement-sample.ts b/tests/dummy/app/controllers/regulatory-statement-sample.ts
@@ -188,34 +188,28 @@ export default class RegulatoryStatementSampleController extends Controller {
     return [
       {
         label: 'text',
-        // @ts-expect-error Unclear why these types don't agree
         component: TextVariableInsertComponent,
       },
       {
         label: 'number',
-        // @ts-expect-error Unclear why these types don't agree
         component: NumberInsertComponent,
       },
       {
         label: 'date',
-        // @ts-expect-error Unclear why these types don't agree
         component: DateInsertVariableComponent,
       },
       {
         label: 'location',
-        // @ts-expect-error Unclear why these types don't agree
         component: LocationInsertComponent,
         options: this.locationOptions,
       },
       {
         label: 'codelist',
-        // @ts-expect-error Unclear why these types don't agree
         component: CodelistInsertComponent,
         options: this.codelistOptions,
       },
       {
         label: 'address',
-        // @ts-expect-error Unclear why these types don't agree
         component: VariablePluginAddressInsertVariableComponent,
       },
     ];