Don't send null fields in app management and clarify the fields
This commit is contained in:
parent
f7c27f250b
commit
10004192d7
2 changed files with 11 additions and 3 deletions
|
@ -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;
|
||||
|
|
|
@ -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">
|
||||
|
|
Reference in a new issue