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

Commit 2a3664c

Browse files
Merge pull request #2100 from github/fixes/annotations-metrics
Adding NumberOfPullRequestOpenAnnotationsList Metric
2 parents 022a42f + 851cbcc commit 2a3664c

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

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

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
using System.Linq;
55
using System.Reactive;
66
using System.Threading.Tasks;
7+
using GitHub.Extensions;
78
using GitHub.Models;
89
using GitHub.Services;
910
using ReactiveUI;
@@ -17,6 +18,7 @@ public class PullRequestAnnotationsViewModel : PanePageViewModelBase, IPullReque
1718
{
1819
readonly IPullRequestSessionManager sessionManager;
1920
readonly IPullRequestEditorService pullRequestEditorService;
21+
readonly IUsageTracker usageTracker;
2022

2123
IPullRequestSession session;
2224
string title;
@@ -32,10 +34,15 @@ public class PullRequestAnnotationsViewModel : PanePageViewModelBase, IPullReque
3234
/// <param name="sessionManager">The pull request session manager.</param>
3335
/// <param name="pullRequestEditorService">The pull request editor service.</param>
3436
[ImportingConstructor]
35-
public PullRequestAnnotationsViewModel(IPullRequestSessionManager sessionManager, IPullRequestEditorService pullRequestEditorService)
37+
public PullRequestAnnotationsViewModel(
38+
IPullRequestSessionManager sessionManager,
39+
IPullRequestEditorService pullRequestEditorService,
40+
IUsageTracker usageTracker)
3641
{
3742
this.sessionManager = sessionManager;
3843
this.pullRequestEditorService = pullRequestEditorService;
44+
this.usageTracker = usageTracker;
45+
3946
NavigateToPullRequest = ReactiveCommand.Create(() => {
4047
NavigateTo(FormattableString.Invariant(
4148
$"{LocalRepository.Owner}/{LocalRepository.Name}/pull/{PullRequestNumber}"));
@@ -158,6 +165,8 @@ void Load(PullRequestDetailModel pullRequest)
158165
.Cast<IPullRequestAnnotationItemViewModel>()
159166
.ToArray()
160167
);
168+
169+
usageTracker.IncrementCounter(x => x.NumberOfPullRequestOpenAnnotationsList).Forget();
161170
}
162171
finally
163172
{

src/GitHub.Exports/Models/UsageModel.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,7 @@ public class MeasuresModel
7575
public int NumberOfShowCurrentPullRequest { get; set; }
7676
public int NumberOfStatusBarOpenPullRequestList { get; set; }
7777
public int NumberOfTeamExplorerHomeOpenPullRequestList { get; set; }
78+
public int NumberOfPullRequestOpenAnnotationsList { get; set; }
7879
public int NumberOfStartPageClones { get; set; }
7980
public int NumberOfGitHubConnectSectionClones { get; set; }
8081
public int NumberOfShowRepoForkDialogClicks { get; set; }

0 commit comments

Comments
 (0)