Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Commit ae7960e

Browse files
committed
Add usage tracking for SyncSubmodules
1 parent fba23fd commit ae7960e

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

src/GitHub.App/ViewModels/GitHubPane/PullRequestDetailViewModel.cs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -657,9 +657,7 @@ IObservable<Unit> DoPush(object unused)
657657

658658
IObservable<Unit> DoSyncSubmodules(object unused)
659659
{
660-
return pullRequestsService.SyncSubmodules(LocalRepository);
661-
// TODO: UsageTracker
662-
// .Do(_ => usageTracker.IncrementPullRequestPushCount(IsFromFork).Forget());
660+
return pullRequestsService.SyncSubmodules(LocalRepository).Do(_ => usageTracker.IncrementCounter(x => x.NumberOfSyncSubmodules).Forget());
663661
}
664662

665663
class CheckoutCommandState : IPullRequestCheckoutState

src/GitHub.Exports/Models/UsageModel.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ public class UsageModel
2828
public int NumberOfForkPullRequestsCheckedOut { get; set; }
2929
public int NumberOfForkPullRequestPulls { get; set; }
3030
public int NumberOfForkPullRequestPushes { get; set; }
31+
public int NumberOfSyncSubmodules { get; set; }
3132
public int NumberOfWelcomeDocsClicks { get; set; }
3233
public int NumberOfWelcomeTrainingClicks { get; set; }
3334
public int NumberOfGitHubPaneHelpClicks { get; set; }

0 commit comments

Comments
 (0)