Merge pull request #422 from github/fixes/401-description-missing

Include description in PR Creation

Author: shana

src/GitHub.App/Services/ModelService.cs

@@ -199,15 +199,15 @@ public ITrackingCollection<IRepositoryModel> GetRepositories(ITrackingCollection
             return collection;
         }
 
-        public IObservable<IPullRequestModel> CreatePullRequest(ISimpleRepositoryModel repository, string title, IBranch source, IBranch target)
+        public IObservable<IPullRequestModel> CreatePullRequest(ISimpleRepositoryModel repository, string title, string body, IBranch source, IBranch target)
         {
             var keyobs = GetUserFromCache()
                 .Select(user => string.Format(CultureInfo.InvariantCulture, "{0}|{1}:{2}", CacheIndex.PRPrefix, user.Login, repository.Name));
 
             return Observable.Defer(() => keyobs
                 .SelectMany(key =>
                     hostCache.PutAndUpdateIndex(key, () =>
-                        apiClient.CreatePullRequest(new NewPullRequest(title, source.Name, target.Name),
+                        apiClient.CreatePullRequest(new NewPullRequest(title, source.Name, target.Name) { Body = body },
                                     repository.CloneUrl.Owner,
                                     repository.CloneUrl.RepositoryName)
                                  .Select(PullRequestCacheItem.Create),

src/GitHub.App/Services/PullRequestService.cs

@@ -8,9 +8,9 @@ namespace GitHub.Services
     [PartCreationPolicy(CreationPolicy.Shared)]
     public class PullRequestService : IPullRequestService
     {
-        public IObservable<IPullRequestModel> CreatePullRequest(IRepositoryHost host, ISimpleRepositoryModel repository, string title, IBranch source, IBranch target)
+        public IObservable<IPullRequestModel> CreatePullRequest(IRepositoryHost host, ISimpleRepositoryModel repository, string title, string body, IBranch source, IBranch target)
         {
-            return host.ModelService.CreatePullRequest(repository, title, source, target);
+            return host.ModelService.CreatePullRequest(repository, title, body, source, target);
         }
     }
 }

src/GitHub.App/ViewModels/PullRequestCreationViewModel.cs

@@ -68,7 +68,7 @@ public PullRequestCreationViewModel(IRepositoryHost repositoryHost, ISimpleRepos
                 .Subscribe(x => notifications.ShowError(BranchValidator.ValidationResult.Message));
 
             createPullRequest = ReactiveCommand.CreateAsyncObservable(whenAnyValidationResultChanges,
-                _ => service.CreatePullRequest(repositoryHost, activeRepo, PRTitle, SourceBranch, TargetBranch)
+                _ => service.CreatePullRequest(repositoryHost, activeRepo, PRTitle, Description, SourceBranch, TargetBranch)
             );
             createPullRequest.ThrownExceptions.Subscribe(ex =>
             {

src/GitHub.Exports.Reactive/Services/IModelService.cs

@@ -20,7 +20,7 @@ public interface IModelService : IDisposable
         IObservable<LicenseItem> GetLicenses();
         IObservable<GitIgnoreItem> GetGitIgnoreTemplates();
         ITrackingCollection<IPullRequestModel> GetPullRequests(ISimpleRepositoryModel repo, ITrackingCollection<IPullRequestModel> collection);
-        IObservable<IPullRequestModel> CreatePullRequest(ISimpleRepositoryModel repository, string title, IBranch source, IBranch target);
+        IObservable<IPullRequestModel> CreatePullRequest(ISimpleRepositoryModel repository, string title, string body, IBranch source, IBranch target);
         IObservable<IBranch> GetBranches(ISimpleRepositoryModel repo);
         IObservable<Unit> InvalidateAll();
     }

src/GitHub.Exports.Reactive/Services/IPullRequestService.cs

@@ -10,6 +10,6 @@ namespace GitHub.Services
 {
     public interface IPullRequestService
     {
-        IObservable<IPullRequestModel> CreatePullRequest(IRepositoryHost host, ISimpleRepositoryModel repository, string title, IBranch source, IBranch target);
+        IObservable<IPullRequestModel> CreatePullRequest(IRepositoryHost host, ISimpleRepositoryModel repository, string title, string body, IBranch source, IBranch target);
     }
 }