vbatts/quay
Archived
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #1576 from coreos-inc/new-repo-error

Browse source 
Fix error display in create repo view
This commit is contained in:
josephschorr 2016-06-24 11:47:39 -04:00 committed by GitHub
commit 31d5ff0649
2 changed files with 9 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
static/js/pages/new-repo.js
View file

@ -53,9 +53,11 @@
return true;
};
$scope.createNewRepo = function() {
$('#repoName').popover('hide');
$scope.$watch('repo.name', function() {
$scope.createError = null;
});
$scope.createNewRepo = function() {
$scope.creating = true;
var repo = $scope.repo;
var data = {
@ -86,10 +88,7 @@
$location.path('/repository/' + created.namespace + '/' + created.name);
}, function(result) {
$scope.creating = false;
$scope.createError = result.data ? result.data.message : 'Cannot create repository';
$timeout(function() {
$('#repoName').popover('show');
});
$scope.createError = ApiService.getErrorMessage(result);
});
};
}

5
static/partials/new-repo.html
View file

@ -41,9 +41,12 @@
<input id="repoName" name="repoName" type="text" class="form-control" placeholder="Repository Name" ng-model="repo.name"
required autofocus data-trigger="manual" data-content="{{ createError }}" data-placement="right" ng-pattern="/^[.a-z0-9_-]+$/">
</span>
<span class="co-alert co-alert-warning co-alert-popin-warning" ng-show="!newRepoForm.repoName.$error.required && !newRepoForm.repoName.$valid">
<span class="co-alert co-alert-warning co-alert-popin-warning" ng-show="!creating && !newRepoForm.repoName.$error.required && !newRepoForm.repoName.$valid">
Repository names must match [a-z0-9_-]+
</span>
<span class="co-alert co-alert-danger co-alert-popin-warning" ng-show="!creating && createError">
{{ createError }}
</span>
</div>
</div>
</div>