Don't show deeplinks for undeployed resources (#1043)

Don't show deeplinks for undeployed resources
databricks · Feb 6, 2024 · 62a47e3 · 62a47e3
1 parent 5b1eb9b
commit 62a47e3
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/packages/databricks-vscode/src/ui/bundle-resource-explorer/JobTreeNode.ts b/packages/databricks-vscode/src/ui/bundle-resource-explorer/JobTreeNode.ts
@@ -18,7 +18,7 @@ export class JobTreeNode implements BundleResourceExplorerTreeNode {
     get url(): string | undefined {
         const host = this.connectionManager.databricksWorkspace?.host;
 
-        if (host === undefined) {
+        if (!host || !this.data.id) {
             return undefined;
         }
 

diff --git a/packages/databricks-vscode/src/ui/bundle-resource-explorer/PipelineTreeNode.ts b/packages/databricks-vscode/src/ui/bundle-resource-explorer/PipelineTreeNode.ts
@@ -17,7 +17,7 @@ export class PipelineTreeNode implements BundleResourceExplorerTreeNode {
     get url(): string | undefined {
         const host = this.connectionManager.databricksWorkspace?.host;
 
-        if (host === undefined) {
+        if (!host || !this.data.id) {
             return undefined;
         }
 

diff --git a/packages/databricks-vscode/src/ui/bundle-resource-explorer/TaskTreeNode.ts b/packages/databricks-vscode/src/ui/bundle-resource-explorer/TaskTreeNode.ts
@@ -29,7 +29,7 @@ export class TaskTreeNode implements BundleResourceExplorerTreeNode {
 
     get url(): string | undefined {
         const host = this.connectionManager.databricksWorkspace?.host;
-        if (host === undefined || this.taskKey === undefined) {
+        if (!host || !this.jobId || !this.taskKey) {
             return undefined;
         }