Hide Create new list button
on Issues and MRs pages
This commit is contained in:
parent
146e905995
commit
5f6c6d6a0d
3 changed files with 43 additions and 3 deletions
|
@ -27,9 +27,7 @@
|
|||
= render "shared/issuable/label_dropdown"
|
||||
|
||||
.pull-right
|
||||
- if controller.controller_name != 'boards'
|
||||
= render 'shared/sort_dropdown'
|
||||
- if can?(current_user, :admin_list, @project)
|
||||
- if controller.controller_name == 'boards' && can?(current_user, :admin_list, @project)
|
||||
.dropdown
|
||||
%button.btn.btn-create.js-new-board-list{ type: "button", data: { toggle: "dropdown", labels: labels_filter_path, project_id: @project.try(:id) } }
|
||||
Create new list
|
||||
|
@ -38,6 +36,8 @@
|
|||
- if can?(current_user, :admin_label, @project)
|
||||
= render partial: "shared/issuable/label_page_create"
|
||||
= dropdown_loading
|
||||
- else
|
||||
= render 'shared/sort_dropdown'
|
||||
|
||||
- if controller.controller_name == 'issues'
|
||||
.issues_bulk_update.hide
|
||||
|
|
20
spec/features/projects/issues/list_spec.rb
Normal file
20
spec/features/projects/issues/list_spec.rb
Normal file
|
@ -0,0 +1,20 @@
|
|||
require 'spec_helper'
|
||||
|
||||
feature 'Issues List' do
|
||||
let(:user) { create(:user) }
|
||||
let(:project) { create(:empty_project) }
|
||||
|
||||
background do
|
||||
project.team << [user, :developer]
|
||||
|
||||
login_as(user)
|
||||
end
|
||||
|
||||
scenario 'user does not see create new list button' do
|
||||
create(:issue, project: project)
|
||||
|
||||
visit namespace_project_issues_path(project.namespace, project)
|
||||
|
||||
expect(page).not_to have_selector('.js-new-board-list')
|
||||
end
|
||||
end
|
20
spec/features/projects/merge_requests/list_spec.rb
Normal file
20
spec/features/projects/merge_requests/list_spec.rb
Normal file
|
@ -0,0 +1,20 @@
|
|||
require 'spec_helper'
|
||||
|
||||
feature 'Merge Requests List' do
|
||||
let(:user) { create(:user) }
|
||||
let(:project) { create(:project) }
|
||||
|
||||
background do
|
||||
project.team << [user, :developer]
|
||||
|
||||
login_as(user)
|
||||
end
|
||||
|
||||
scenario 'user does not see create new list button' do
|
||||
create(:merge_request, source_project: project)
|
||||
|
||||
visit namespace_project_merge_requests_path(project.namespace, project)
|
||||
|
||||
expect(page).not_to have_selector('.js-new-board-list')
|
||||
end
|
||||
end
|
Loading…
Reference in a new issue