Commit 4454470d authored by Clement Ho's avatar Clement Ho

Remove duplicate delete button in epic

parent 455ffc96
---
title: Remove duplicate delete button in epic
merge_request:
author:
type: fixed
...@@ -121,6 +121,7 @@ ...@@ -121,6 +121,7 @@
:project-path="projectPath" :project-path="projectPath"
:project-namespace="projectNamespace" :project-namespace="projectNamespace"
:show-inline-edit-button="true" :show-inline-edit-button="true"
:show-delete-button="false"
/> />
</div> </div>
<epic-sidebar <epic-sidebar
...@@ -135,7 +136,6 @@ ...@@ -135,7 +136,6 @@
:project-path="projectPath" :project-path="projectPath"
:project-namespace="projectNamespace" :project-namespace="projectNamespace"
:show-inline-edit-button="true" :show-inline-edit-button="true"
:show-delete-button="false"
issuable-type="epic" issuable-type="epic"
:editable="canUpdate" :editable="canUpdate"
:initial-start-date="startDate" :initial-start-date="startDate"
......
...@@ -37,4 +37,18 @@ feature 'Update Epic', :js do ...@@ -37,4 +37,18 @@ feature 'Update Epic', :js do
expect(find('.issuable-details .description')).to have_content('New epic description') expect(find('.issuable-details .description')).to have_content('New epic description')
end end
end end
context 'when user with owner access displays the epic' do
before do
group.add_owner(user)
visit group_epic_path(group, epic)
wait_for_requests
end
it 'does not show delete button inside the edit form' do
find('.btn-edit').click
expect(page).not_to have_selector('.issuable-details .btn-danger')
end
end
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