Merge pull request #1837 from github/fixes/1683-pr-authoring-wrong-fork

StanleyGoldman · web-flow · commit 3bdc45ea31ed · 2018-08-07T11:47:31.000-04:00
Navigate to the correct fork in PR authoring.
diff --git a/src/GitHub.App/ViewModels/GitHubPane/PullRequestReviewAuthoringViewModel.cs b/src/GitHub.App/ViewModels/GitHubPane/PullRequestReviewAuthoringViewModel.cs
@@ -68,7 +68,7 @@ public PullRequestReviewAuthoringViewModel(
                 _ => DoSubmit(Octokit.PullRequestReviewEvent.RequestChanges));
             Cancel = ReactiveCommand.CreateAsyncTask(DoCancel);
             NavigateToPullRequest = ReactiveCommand.Create().OnExecuteCompleted(_ =>
-                NavigateTo(Invariant($"{LocalRepository.Owner}/{LocalRepository.Name}/pull/{PullRequestModel.Number}")));
+                NavigateTo(Invariant($"{RemoteRepositoryOwner}/{LocalRepository.Name}/pull/{PullRequestModel.Number}")));
         }
 
         /// <inheritdoc/>

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ public PullRequestReviewAuthoringViewModel(`
`68`	`68`	`_ => DoSubmit(Octokit.PullRequestReviewEvent.RequestChanges));`
`69`	`69`	`Cancel = ReactiveCommand.CreateAsyncTask(DoCancel);`
`70`	`70`	`NavigateToPullRequest = ReactiveCommand.Create().OnExecuteCompleted(_ =>`
`71`		`- NavigateTo(Invariant($"{LocalRepository.Owner}/{LocalRepository.Name}/pull/{PullRequestModel.Number}")));`
	`71`	`+ NavigateTo(Invariant($"{RemoteRepositoryOwner}/{LocalRepository.Name}/pull/{PullRequestModel.Number}")));`
`72`	`72`	`}`
`73`	`73`
`74`	`74`	`/// <inheritdoc/>`