From 4abe87f99d62eb0a89ff7b9d08c2667126deedb5 Mon Sep 17 00:00:00 2001
From: Samuel Imolorhe <samuelimolo4real@gmail.com>
Date: Sat, 16 Dec 2023 19:42:19 +0100
Subject: [PATCH] Fixed missing schema form input for lists and strings

---
 .../schema-form-item-input.component.ts           |  2 +-
 .../schema-form-item-list.component.ts            |  2 +-
 .../schema-form-item.component.html               |  4 ++--
 .../schema-form-item.component.ts                 | 15 +++++++++++++++
 .../.vuepress/theme/components/Contributions.vue  |  2 --
 yarn.lock                                         | 15 +++++----------
 6 files changed, 24 insertions(+), 16 deletions(-)

diff --git a/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item-input/schema-form-item-input.component.ts b/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item-input/schema-form-item-input.component.ts
index 0d1c38db18..02b8b1934a 100644
--- a/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item-input/schema-form-item-input.component.ts
+++ b/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item-input/schema-form-item-input.component.ts
@@ -9,7 +9,7 @@ import { ThemeRegistryService } from '../../../services';
 })
 export class SchemaFormItemInputComponent {
   @Input() item?: SchemaFormProperty;
-  @Input() data = '';
+  @Input() data: string | undefined;
 
   @Output() dataChange = new EventEmitter();
 
diff --git a/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item-list/schema-form-item-list.component.ts b/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item-list/schema-form-item-list.component.ts
index 341b37837f..15e525e3cc 100644
--- a/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item-list/schema-form-item-list.component.ts
+++ b/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item-list/schema-form-item-list.component.ts
@@ -13,7 +13,7 @@ import { JSONSchema6 } from 'json-schema';
 })
 export class SchemaFormItemListComponent {
   @Input() item?: SchemaFormProperty;
-  @Input() data: unknown[] = [];
+  @Input() data: unknown[] | undefined;
 
   @Output() dataChange = new EventEmitter();
 
diff --git a/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item/schema-form-item.component.html b/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item/schema-form-item.component.html
index e9c9eaac4a..d649c9f85f 100644
--- a/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item/schema-form-item.component.html
+++ b/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item/schema-form-item.component.html
@@ -22,7 +22,7 @@
     </ng-container>
     <ng-container *ngSwitchCase="'string'">
       <app-schema-form-item-input
-        *ngIf="isString(data)"
+        *ngIf="isStringOrUndefined(data)"
         [item]="item"
         [(data)]="data"
         (dataChange)="onInput($event, item)"
@@ -30,7 +30,7 @@
     </ng-container>
     <ng-container *ngSwitchCase="'array'">
       <app-schema-form-item-list
-        *ngIf="isArray(data)"
+        *ngIf="isArrayOrUndefined(data)"
         [item]="item"
         [(data)]="data"
         (dataChange)="onInput($event, item)"
diff --git a/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item/schema-form-item.component.ts b/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item/schema-form-item.component.ts
index 6a53fcdade..829cdab529 100644
--- a/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item/schema-form-item.component.ts
+++ b/packages/altair-app/src/app/modules/altair/components/schema-form/schema-form-item/schema-form-item.component.ts
@@ -63,10 +63,25 @@ export class SchemaFormItemComponent {
     return typeof data === 'string';
   }
 
+  isStringOrUndefined(data: unknown): data is string | undefined {
+    return typeof data === 'string' || typeof data === 'undefined';
+  }
+
   isArray(data: unknown): data is unknown[] {
     return Array.isArray(data);
   }
 
+  isArrayOrUndefined(data: unknown): data is unknown[] | undefined {
+    return Array.isArray(data) || typeof data === 'undefined';
+  }
+
+  asArrayOrUndefined(data: unknown): unknown[] | undefined {
+    if (Array.isArray(data)) {
+      return data;
+    }
+    return undefined;
+  }
+
   onInput(event: Event, item: SchemaFormProperty) {
     // console.log(event, item);
     this.dataChange.next(this.data);
diff --git a/site/src/.vuepress/theme/components/Contributions.vue b/site/src/.vuepress/theme/components/Contributions.vue
index 0196db7cf0..f0d3fe421d 100644
--- a/site/src/.vuepress/theme/components/Contributions.vue
+++ b/site/src/.vuepress/theme/components/Contributions.vue
@@ -1,6 +1,4 @@
 <template>
-  <iframe src="https://altairgraphql.substack.com/embed" width="480" height="320" style="border:1px solid #EEE; background:var(--theme-bg-color);" frameborder="0" scrolling="no"></iframe>
-
   <div class="contributions">
 
     <template
diff --git a/yarn.lock b/yarn.lock
index be643f5645..8841866d3a 100644
--- a/yarn.lock
+++ b/yarn.lock
@@ -22934,10 +22934,10 @@ oauth-sign@~0.9.0:
   resolved "https://registry.yarnpkg.com/oauth-sign/-/oauth-sign-0.9.0.tgz#47a7b016baa68b5fa0ecf3dee08a85c679ac6455"
   integrity sha512-fexhUFFPTGV8ybAtSIGbV6gOkSv8UtRbDBnAyLQw4QPKkgNlsH2ByPGtMUqdWkos6YCRmAqViwgZrJc/mRDzZQ==
 
-oauth@0.9.x:
-  version "0.9.15"
-  resolved "https://registry.yarnpkg.com/oauth/-/oauth-0.9.15.tgz#bd1fefaf686c96b75475aed5196412ff60cfb9c1"
-  integrity sha512-a5ERWK1kh38ExDEfoO6qUHJb32rd7aYmPHuyCu3Fta/cnICvYmgd2uhuKXvPD+PXB+gCEYYEaQdIRAjCOwAKNA==
+oauth@0.10.0, oauth@0.9.x:
+  version "0.10.0"
+  resolved "https://registry.yarnpkg.com/oauth/-/oauth-0.10.0.tgz#3551c4c9b95c53ea437e1e21e46b649482339c58"
+  integrity sha512-1orQ9MT1vHFGQxhuy7E/0gECD3fd2fCC+PIX+/jgmU/gI3EpRocXtmtvxCO5x3WZ443FLTLFWNDjl5MPJf9u+Q==
 
 object-assign@^4, object-assign@^4.0.1, object-assign@^4.1.0, object-assign@^4.1.1:
   version "4.1.1"
@@ -30298,16 +30298,11 @@ xxhashjs@~0.2.2:
   dependencies:
     cuint "^0.2.2"
 
-y18n@^4.0.0:
+y18n@4.0.1, y18n@^4.0.0, y18n@^5.0.5:
   version "4.0.1"
   resolved "https://registry.yarnpkg.com/y18n/-/y18n-4.0.1.tgz#8db2b83c31c5d75099bb890b23f3094891e247d4"
   integrity sha512-wNcy4NvjMYL8gogWWYAO7ZFWFfHcbdbE57tZO8e4cbpj8tfUcwrwqSl3ad8HxpYWCdXcJUCeKKZS62Av1affwQ==
 
-y18n@^5.0.5:
-  version "5.0.8"
-  resolved "https://registry.yarnpkg.com/y18n/-/y18n-5.0.8.tgz#7f4934d0f7ca8c56f95314939ddcd2dd91ce1d55"
-  integrity sha512-0pfFzegeDWJHJIAmTLRP2DwHjdF5s7jo9tuztdQxAhINCdvS+3nGINqPd00AphqJR/0LhANUS6/+7SCb98YOfA==
-
 yaeti@^0.0.6:
   version "0.0.6"
   resolved "https://registry.yarnpkg.com/yaeti/-/yaeti-0.0.6.tgz#f26f484d72684cf42bedfb76970aa1608fbf9577"