Skip to content

Commit e3710cb

Browse files
authored
fix - update provider selection in chat setup (microsoft#254819)
1 parent adb4129 commit e3710cb

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/vs/workbench/contrib/chat/browser/chatSetup.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1255,7 +1255,7 @@ class ChatSetupController extends Disposable {
12551255
this.setStep(ChatSetupStep.SigningIn);
12561256
const result = await this.signIn({ useAlternateProvider: options.useAlternateProvider });
12571257
if (!result.session) {
1258-
const provider = options.useAlternateProvider ? defaultChat.alternativeProviderId : options.useEnterpriseProvider ? defaultChat.enterpriseProviderId : defaultChat.providerName;
1258+
const provider = options.useAlternateProvider ? defaultChat.alternativeProviderId : options.useEnterpriseProvider ? defaultChat.enterpriseProviderId : defaultChat.providerId;
12591259
this.telemetryService.publicLog2<InstallChatEvent, InstallChatClassification>('commandCenter.chatInstall', { installResult: 'failedNotSignedIn', installDuration: watch.elapsed(), signUpErrorCode: undefined, provider });
12601260
return undefined; // treat as cancelled because signing in already triggers an error dialog
12611261
}
@@ -1304,7 +1304,7 @@ class ChatSetupController extends Disposable {
13041304
const wasRunning = this.context.state.installed && !this.context.state.disabled;
13051305
let signUpResult: boolean | { errorCode: number } | undefined = undefined;
13061306

1307-
const provider = options.useAlternateProvider ? defaultChat.alternativeProviderId : options.useEnterpriseProvider ? defaultChat.enterpriseProviderId : defaultChat.providerName;
1307+
const provider = options.useAlternateProvider ? defaultChat.alternativeProviderId : options.useEnterpriseProvider ? defaultChat.enterpriseProviderId : defaultChat.providerId;
13081308

13091309
try {
13101310

0 commit comments

Comments
 (0)