Skip to content

Commit 1c16780

Browse files
luabudkarthiknadig
andauthored
Replace machine ID from survey with session ID (#11844)
* Replace machin ID from survey with session ID * Removing space from open brackets * Fix tests * Undo launch.json changes Co-authored-by: Karthik Nadig <[email protected]>
1 parent 0e59e32 commit 1c16780

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

src/client/activation/extensionSurvey.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ export class ExtensionSurveyPrompt implements IExtensionSingleActivationService
114114
o: encodeURIComponent(this.platformService.osType), // platform
115115
v: encodeURIComponent(this.appEnvironment.vscodeVersion),
116116
e: encodeURIComponent(this.appEnvironment.packageJson.version), // extension version
117-
m: encodeURIComponent(this.appEnvironment.machineId)
117+
m: encodeURIComponent(this.appEnvironment.sessionId)
118118
});
119119
const url = `https://aka.ms/AA5rjx5?${query}`;
120120
this.browserService.launch(url);

src/test/activation/extensionSurvey.unit.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,7 @@ suite('Extension survey prompt - showSurvey()', () => {
227227
version: 'extensionVersion'
228228
};
229229
const prompts = [LanguageService.bannerLabelYes(), ExtensionSurveyBanner.maybeLater(), Common.doNotShowAgain()];
230-
const expectedUrl = `https://aka.ms/AA5rjx5?o=Windows&v=vscodeVersion&e=extensionVersion&m=machineId`;
230+
const expectedUrl = `https://aka.ms/AA5rjx5?o=Windows&v=vscodeVersion&e=extensionVersion&m=sessionId`;
231231
appEnvironment
232232
.setup((a) => a.packageJson)
233233
.returns(() => packageJson)
@@ -237,8 +237,8 @@ suite('Extension survey prompt - showSurvey()', () => {
237237
.returns(() => 'vscodeVersion')
238238
.verifiable(TypeMoq.Times.once());
239239
appEnvironment
240-
.setup((a) => a.machineId)
241-
.returns(() => 'machineId')
240+
.setup((a) => a.sessionId)
241+
.returns(() => 'sessionId')
242242
.verifiable(TypeMoq.Times.once());
243243
platformService
244244
.setup((a) => a.osType)

0 commit comments

Comments
 (0)