From 1caef2dc6ee5cbac47fe1d7907b7f99e0b1d860d Mon Sep 17 00:00:00 2001 From: Charles Gagnon Date: Thu, 24 Jun 2021 23:55:53 -0700 Subject: [PATCH] Remove duplicate extensions from browse filter (#15907) * Remove duplicate extensions from browse filter * Update placeholder --- extensions/data-workspace/src/common/constants.ts | 2 +- extensions/data-workspace/src/dialogs/openExistingDialog.ts | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/extensions/data-workspace/src/common/constants.ts b/extensions/data-workspace/src/common/constants.ts index 9c54310b33..ff9fa779c5 100644 --- a/extensions/data-workspace/src/common/constants.ts +++ b/extensions/data-workspace/src/common/constants.ts @@ -64,7 +64,7 @@ export const CloneParentDirectoryNotExistError = (location: string): string => { export const Project = localize('dataworkspace.project', "Project"); export const Workspace = localize('dataworkspace.workspace', "Workspace"); export const LocationSelectorTitle = localize('dataworkspace.locationSelectorTitle', "Location"); -export const ProjectFilePlaceholder = localize('dataworkspace.projectFilePlaceholder', "Select project (.sqlproj) file"); +export const ProjectFilePlaceholder = localize('dataworkspace.projectFilePlaceholder', "Select project file"); export const WorkspacePlaceholder = localize('dataworkspace.workspacePlaceholder', "Select workspace ({0}) file", WorkspaceFileExtension); export const ProjectAlreadyOpened = (path: string): string => { return localize('dataworkspace.projectAlreadyOpened', "Project '{0}' is already opened.", path); }; export const Local = localize('dataworksapce.local', 'Local'); diff --git a/extensions/data-workspace/src/dialogs/openExistingDialog.ts b/extensions/data-workspace/src/dialogs/openExistingDialog.ts index badef16dcf..ba88becfe2 100644 --- a/extensions/data-workspace/src/dialogs/openExistingDialog.ts +++ b/extensions/data-workspace/src/dialogs/openExistingDialog.ts @@ -374,7 +374,7 @@ export class OpenExistingDialog extends DialogBase { public async projectBrowse(): Promise { const filters: { [name: string]: string[] } = {}; const projectTypes = await this.workspaceService.getAllProjectTypes(); - filters[constants.AllProjectTypes] = projectTypes.map(type => type.projectFileExtension); + filters[constants.AllProjectTypes] = [...new Set(projectTypes.map(type => type.projectFileExtension))]; projectTypes.forEach(type => { filters[type.displayName] = [type.projectFileExtension]; });