Merge pull request #420 from BalticAmadeus/398-delete-button-warning

Add connection deletion confirmation.
BalticAmadeus · Oct 4, 2023 · 5a4967d · 5a4967d
2 parents 5f64db3 + c610699
commit 5a4967d
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/src/extension.ts b/src/extension.ts
@@ -270,8 +270,17 @@ export function activate(context: vscode.ExtensionContext) {
   context.subscriptions.push(
     vscode.commands.registerCommand(
       `${Constants.globalExtensionKey}.deleteConnection`,
-      (dbConnectionNode: DbConnectionNode) => {
-        dbConnectionNode.deleteConnection(context);
+      async (dbConnectionNode: DbConnectionNode) => {
+        const confirmation = await vscode.window.showWarningMessage(
+          `Are you sure you want to delete the connection "${dbConnectionNode.config.label}"?`,
+          { modal: true },
+          'Yes',
+          'No'
+        );
+
+        if (confirmation === 'Yes') {
+          dbConnectionNode.deleteConnection(context);
+        }
       }
     )
   );