Skip to content

Commit 92d2586

Browse files
Updates telemetry even name
1 parent 47373f4 commit 92d2586

File tree

4 files changed

+9
-9
lines changed

4 files changed

+9
-9
lines changed

docs/telemetry-events.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1582,9 +1582,9 @@ or
15821582
}
15831583
```
15841584

1585-
### composer/opened
1585+
### composer/loaded
15861586

1587-
> Sent when the Commit Composer is opened
1587+
> Sent when the Commit Composer is first loaded with repo data
15881588
15891589
```typescript
15901590
{

src/constants.telemetry.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,8 +129,8 @@ export interface TelemetryEvents extends WebviewShowAbortedEvents, WebviewShownE
129129
/** Sent when the user changes the selected tab (mode) on the Graph Details view */
130130
'commitDetails/mode/changed': CommitDetailsModeChangedEvent;
131131

132-
/** Sent when the Commit Composer is opened */
133-
'composer/opened': ComposerEvent;
132+
/** Sent when the Commit Composer is first loaded with repo data */
133+
'composer/loaded': ComposerEvent;
134134
/** Sent when the Commit Composer is reloaded */
135135
'composer/reloaded': ComposerEvent;
136136
/** Sent when the user adds unstaged changes to draft commits in the Commit Composer */

src/webviews/plus/composer/composerWebview.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -289,7 +289,7 @@ export class ComposerWebviewProvider implements WebviewProvider<State, State, Co
289289
const baseCommit = await repo.git.commits.getCommit('HEAD');
290290
if (baseCommit == null) {
291291
const errorMessage = 'No base commit found to compose from.';
292-
this.sendTelemetryEvent(isReload ? 'composer/reloaded' : 'composer/opened', {
292+
this.sendTelemetryEvent(isReload ? 'composer/reloaded' : 'composer/loaded', {
293293
'failure.reason': 'error',
294294
'failure.error.message': errorMessage,
295295
});
@@ -302,7 +302,7 @@ export class ComposerWebviewProvider implements WebviewProvider<State, State, Co
302302
const currentBranch = await repo.git.branches.getBranch();
303303
if (currentBranch == null) {
304304
const errorMessage = 'No current branch found to compose from.';
305-
this.sendTelemetryEvent(isReload ? 'composer/reloaded' : 'composer/opened', {
305+
this.sendTelemetryEvent(isReload ? 'composer/reloaded' : 'composer/loaded', {
306306
'failure.reason': 'error',
307307
'failure.error.message': errorMessage,
308308
});
@@ -361,7 +361,7 @@ export class ComposerWebviewProvider implements WebviewProvider<State, State, Co
361361
this._context.warnings.workingDirectoryChanged = false;
362362
this._context.warnings.indexChanged = false;
363363
this._context.sessionStart = new Date().toISOString();
364-
this.sendTelemetryEvent(isReload ? 'composer/reloaded' : 'composer/opened');
364+
this.sendTelemetryEvent(isReload ? 'composer/reloaded' : 'composer/loaded');
365365

366366
// Subscribe to repository changes for working directory monitoring
367367
this.subscribeToRepository(repo);
@@ -1132,7 +1132,7 @@ export class ComposerWebviewProvider implements WebviewProvider<State, State, Co
11321132
event: 'composer/action/finishAndCommit/failed',
11331133
data: ComposerActionEventFailureData,
11341134
): void;
1135-
private sendTelemetryEvent(event: 'composer/opened' | 'composer/reloaded', data?: ComposerLoadedErrorData): void;
1135+
private sendTelemetryEvent(event: 'composer/loaded' | 'composer/reloaded', data?: ComposerLoadedErrorData): void;
11361136
private sendTelemetryEvent(
11371137
event:
11381138
| 'composer/action/includedUnstagedChanges'

src/webviews/plus/composer/protocol.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -294,7 +294,7 @@ export const baseContext: ComposerContext = {
294294
};
295295

296296
export type ComposerTelemetryEvent =
297-
| 'composer/opened'
297+
| 'composer/loaded'
298298
| 'composer/reloaded'
299299
| 'composer/action/includedUnstagedChanges'
300300
| 'composer/action/compose'

0 commit comments

Comments
 (0)