From 10004192d79af1a972d86c18f10142e4dfa37f52 Mon Sep 17 00:00:00 2001
From: Joseph Schorr <jschorr@gmail.com>
Date: Mon, 24 Mar 2014 18:18:35 -0400
Subject: [PATCH] Don't send null fields in app management and clarify the
 fields

---
 static/js/controllers.js                | 8 ++++++++
 static/partials/manage-application.html | 6 +++---
 2 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/static/js/controllers.js b/static/js/controllers.js
index 3d186ce88..dad3c47c1 100644
--- a/static/js/controllers.js
+++ b/static/js/controllers.js
@@ -2484,6 +2484,14 @@ function ManageApplicationCtrl($scope, $routeParams, $rootScope, $location, $tim
       'client_id': clientId
     };
 
+    if (!$scope.application['description']) {
+      delete $scope.application['description'];
+    }
+
+    if (!$scope.application['gravatar_email']) {
+      delete $scope.application['gravatar_email'];
+    }
+
     ApiService.updateOrganizationApplication($scope.application, params).then(function(resp) {
       $scope.application = resp;
       $scope.updating = false;
diff --git a/static/partials/manage-application.html b/static/partials/manage-application.html
index 6741aa67d..f14aa7a1b 100644
--- a/static/partials/manage-application.html
+++ b/static/partials/manage-application.html
@@ -61,13 +61,13 @@
               <div class="form-group nested">
                 <label for="fieldAppGravatar">Gravatar E-mail (optional)</label>
                 <input type="email" class="form-control" id="fieldAppGravatar" placeholder="Gravatar E-mail" ng-model="application.gravatar_email">
-                <div class="description">An e-mail address representing the  <a href="http://en.gravatar.com/" target="_blank">Gravatar</a> for the application</div>
+                <div class="description">An e-mail address representing the  <a href="http://en.gravatar.com/" target="_blank">Gravatar</a> for the application. See above for the icon.</div>
               </div>
 
               <div class="form-group nested" style="margin-top: 10px; padding-top: 10px; border-top: 1px solid #eee;">
-                <label for="fieldAppRedirect">Redirect/Callback URL</label>
+                <label for="fieldAppRedirect">Redirect/Callback URL Prefix</label>
                 <input type="url" class="form-control" id="fieldAppRedirect" placeholder="OAuth Redirect URL" ng-model="application.redirect_uri" required>
-                <div class="description">The application's OAuth redirection/callback URL, invoked once access has been granted.</div>
+                <div class="description">Allowed prefix for the application's OAuth redirection/callback URLs</div>
               </div>
 
               <div class="button-bar">