Merge branch 'fix-ci-permissions' into 'master'

Fix CI permissions

![Screen_Shot_2015-09-21_at_20.23.16](https://gitlab.com/gitlab-org/gitlab-ce/uploads/6f40699507d0dce148d0054a093f0d63/Screen_Shot_2015-09-21_at_20.23.16.png)

This fixes two problems:
- Allow developers to retry builds
- Hide advanced project options from CI page for non-admin users



See merge request !1379
This commit is contained in:
Dmitriy Zaporozhets 2015-09-22 07:54:11 +00:00
commit 8ae026a6e6
3 changed files with 45 additions and 42 deletions

View file

@ -44,6 +44,8 @@ v 8.0.0 (unreleased)
- Retrieving oauth token with LDAP credentials
- Load Application settings from running database unless env var USE_DB=false
- Added Drone CI integration (Kirill Zaitsev)
- Allow developers to retry builds
- Hide advanced project options for non-admin users
- Fail builds if no .gitlab-ci.yml is found
- Refactored service API and added automatically service docs generator (Kirill Zaitsev)
- Added web_url key project hook_attrs (Kirill Zaitsev)

View file

@ -38,7 +38,7 @@ module Ci
end
def authorize_manage_builds!
unless can?(current_user, :admin_project, gl_project)
unless can?(current_user, :manage_builds, gl_project)
return page_404
end
end

View file

@ -10,6 +10,7 @@
%span
Commits
%span.count= @project.commits.count
- if can?(current_user, :admin_project, gl_project)
= nav_link path: 'charts#show' do
= link_to ci_project_charts_path(@project) do
= icon('bar-chart fw')