Commit 7cd20b91 authored by Doug Stull's avatar Doug Stull

Allow developers to see ci minutes notifications

- we want to broaden the audience as much as
  we can.
parent 6a67fe97
...@@ -91,7 +91,6 @@ module EE ...@@ -91,7 +91,6 @@ module EE
enable :create_jira_connect_subscription enable :create_jira_connect_subscription
enable :maintainer_access enable :maintainer_access
enable :admin_wiki enable :admin_wiki
enable :read_ci_minutes_quota
end end
rule { owner }.policy do rule { owner }.policy do
...@@ -174,6 +173,7 @@ module EE ...@@ -174,6 +173,7 @@ module EE
rule { developer }.policy do rule { developer }.policy do
enable :create_wiki enable :create_wiki
enable :admin_merge_request enable :admin_merge_request
enable :read_ci_minutes_quota
end end
rule { security_dashboard_enabled & developer }.enable :read_group_security_dashboard rule { security_dashboard_enabled & developer }.enable :read_group_security_dashboard
......
---
title: Allow developers to see ci minutes notifications
merge_request: 31937
author:
type: changed
...@@ -53,7 +53,7 @@ describe Ci::Minutes::Context do ...@@ -53,7 +53,7 @@ describe Ci::Minutes::Context do
describe '#can_see_status' do describe '#can_see_status' do
context 'when eligible to see status' do context 'when eligible to see status' do
before do before do
group.add_owner(user) group.add_developer(user)
end end
it 'can see status' do it 'can see status' do
......
...@@ -128,7 +128,7 @@ describe Ci::Minutes::Notification do ...@@ -128,7 +128,7 @@ describe Ci::Minutes::Notification do
describe '#show?' do describe '#show?' do
context 'when eligible to see notifications' do context 'when eligible to see notifications' do
before do before do
group.add_owner(user) group.add_developer(user)
end end
context 'with a project that has runners enabled inside namespace' do context 'with a project that has runners enabled inside namespace' do
......
...@@ -885,7 +885,7 @@ describe GroupPolicy do ...@@ -885,7 +885,7 @@ describe GroupPolicy do
where(:role, :allowed) do where(:role, :allowed) do
:guest | false :guest | false
:reporter | false :reporter | false
:developer | false :developer | true
:maintainer | true :maintainer | true
:owner | true :owner | true
:admin | true :admin | true
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment