only show confirmation message for apply all changes, not for view changes in schema compare (#18520)

This commit is contained in:
Kim Santiago
2022-02-22 13:33:24 -08:00
committed by GitHub
parent 5547c2baba
commit 7b92c9eb65
2 changed files with 71 additions and 71 deletions

View File

@@ -1382,6 +1382,8 @@ export class ProjectsController {
if (model.action === UpdateProjectAction.Compare) {
await vscode.commands.executeCommand(constants.schemaCompareRunComparisonCommand, model.sourceEndpointInfo, model.targetEndpointInfo, true, undefined);
} else if (model.action === UpdateProjectAction.Update) {
await vscode.window.showWarningMessage(constants.applyConfirmation, { modal: true }, constants.yesString).then(async (result) => {
if (result === constants.yesString) {
await vscode.window.withProgress(
{
location: vscode.ProgressLocation.Notification,
@@ -1393,6 +1395,8 @@ export class ProjectsController {
void vscode.commands.executeCommand(constants.refreshDataWorkspaceCommand);
utils.getDataWorkspaceExtensionApi().showProjectsView();
}
});
} else {
throw new Error(`Unknown UpdateProjectAction: ${model.action}`);
}

View File

@@ -497,8 +497,6 @@ export class UpdateProjectFromDatabaseDialog {
}
public async handleUpdateButtonClick(): Promise<void> {
await vscode.window.showWarningMessage(constants.applyConfirmation, { modal: true }, constants.yesString).then(async (result) => {
if (result === constants.yesString) {
const serverDropdownValue = this.serverDropdown!.value! as azdata.CategoryValue as ConnectionDropdownValue;
const ownerUri = await getAzdataApi()!.connection.getUriForConnection(serverDropdownValue.connection.connectionId);
@@ -560,11 +558,9 @@ export class UpdateProjectFromDatabaseDialog {
targetEndpointInfo: targetEndpointInfo,
action: this.action!
};
void this.updateProjectFromDatabaseCallback!(model);
}
void this.updateProjectFromDatabaseCallback!(model);
this.dispose();
});
}
async validate(): Promise<boolean> {