[frontend] Fix type issues in editor v2 Vue components

cloudera · May 27, 2021 · dea2451 · dea2451
1 parent e55c876
commit dea2451
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/desktop/core/src/desktop/js/apps/editor/components/ExecutionStatusIcon.vue b/desktop/core/src/desktop/js/apps/editor/components/ExecutionStatusIcon.vue
@@ -37,7 +37,7 @@
     name: 'ExecutionStatusIcon',
     props: {
       status: {
-        type: Object as PropType<ExecutionStatus | null>,
+        type: String as PropType<ExecutionStatus | null>,
         default: null
       }
     },

diff --git a/...top/js/apps/editor/components/aceEditor/autocomplete/details/CatalogEntryDetailsPanel.vue b/...top/js/apps/editor/components/aceEditor/autocomplete/details/CatalogEntryDetailsPanel.vue
@@ -50,7 +50,7 @@
         </div>
 
         <div v-if="loading" class="details-comment">
-          <spinner size="small" inline="true" />
+          <spinner size="small" :inline="true" />
         </div>
         <div v-else-if="comment" class="details-comment">{{ comment }}</div>
         <div v-else class="details-no-comment">{{ I18n('No description') }}</div>

diff --git a/desktop/core/src/desktop/js/apps/editor/components/result/ResultTable.vue b/desktop/core/src/desktop/js/apps/editor/components/result/ResultTable.vue
@@ -75,8 +75,8 @@
     },
     props: {
       executable: {
-        type: Object as PropType<Executable>,
-        required: true
+        type: Object as PropType<Executable | undefined>,
+        default: undefined
       }
     },
     setup(props) {
@@ -204,7 +204,7 @@
         if (streaming.value) {
           return;
         }
-        if (hasMore.value && !grayedOut.value && executable.value && executable.value.result) {
+        if (hasMore.value && !grayedOut.value && executable.value?.result) {
           grayedOut.value = true;
           try {
             await executable.value.result.fetchRows({ rows: 100 });