Commit b1da8f0f authored by Sean McGivern's avatar Sean McGivern

Merge branch 'group-memebrs-owner-level' into 'master'

Added ability to change user permissions in group to owner

Closes #28233

See merge request !9299
parents 54439315 3894ae3b
...@@ -61,7 +61,7 @@ ...@@ -61,7 +61,7 @@
= dropdown_title("Change permissions") = dropdown_title("Change permissions")
.dropdown-content .dropdown-content
%ul %ul
- Gitlab::Access.options.each do |role, role_id| - member.class.access_level_roles.each do |role, role_id|
%li %li
= link_to role, "javascript:void(0)", = link_to role, "javascript:void(0)",
class: ("is-active" if member.access_level == role_id), class: ("is-active" if member.access_level == role_id),
......
---
title: Added option to update to owner for group members
merge_request:
author:
...@@ -30,6 +30,21 @@ feature 'Groups members list', feature: true do ...@@ -30,6 +30,21 @@ feature 'Groups members list', feature: true do
expect(second_row).to be_blank expect(second_row).to be_blank
end end
it 'updates user to owner level', :js do
group.add_owner(user1)
group.add_developer(user2)
visit group_group_members_path(group)
page.within(second_row) do
click_button('Developer')
click_link('Owner')
expect(page).to have_button('Owner')
end
end
def first_row def first_row
page.all('ul.content-list > li')[0] page.all('ul.content-list > li')[0]
end end
......
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