Skip to content

Commit

Permalink
Removing the "Withdraw" and "Deny" buttons (#1096)
Browse files Browse the repository at this point in the history
  • Loading branch information
jrgriffiniii authored Dec 12, 2024
1 parent ba97619 commit 7a2f7b4
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 7 deletions.
2 changes: 0 additions & 2 deletions app/views/projects/show.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,6 @@
<div class="details">
<% if !current_user.eligible_sysadmin? %>
<%= link_to "Review Contents", project_contents_path(@project.id), class: "btn btn-primary btn-sm" %>
<%= link_to "Withdraw Project Request", "", class: "btn btn-secondary btn-sm" %>
<%= link_to "Return to Dashboard", root_path, class: "btn btn-primary btn-sm" %>
<% end %>
</div>
Expand All @@ -101,7 +100,6 @@
<% if current_user.eligible_sysadmin? %>
<% if @project.pending? %>
<%= link_to "Approve Project", project_approve_path(@project.id), class: "btn btn-primary btn-sm" %>
<%= link_to "Deny Project", "", class: "btn btn-secondary btn-sm" %>
<%= link_to " View Create Script", "#", class: "bi bi-code btn btn-secondary btn-sm", id: "create-script-btn" %>
<% end %>
<%= link_to "Review Contents", project_contents_path(@project.id), class: "btn btn-primary btn-sm" %>
Expand Down
5 changes: 0 additions & 5 deletions spec/system/project_show_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,6 @@
# Per ticket #1114 sponsor users no longer have edit access
expect(page).not_to have_selector(:link_or_button, "Edit") # button next to role and description heading
expect(page).to have_selector(:link_or_button, "Review Contents")
expect(page).to have_selector(:link_or_button, "Withdraw Project Request")
expect(page).to have_selector(:link_or_button, "Return to Dashboard")
click_on("Return to Dashboard")
expect(page).to have_content("Welcome, #{sponsor_user.given_name}!")
Expand Down Expand Up @@ -93,7 +92,6 @@
click_on("Return to Dashboard")
expect(page).to have_content("Welcome, #{sponsor_user.given_name}!")
find(:xpath, "//h2[text()='#{project_in_mediaflux.title}']").click
expect(page).to have_link("Withdraw Project Request")
end
end
context "SysAdmin" do
Expand All @@ -105,7 +103,6 @@
expect(page).to have_css ".pending"
expect(page).not_to have_link("Edit")
expect(page).to have_selector(:link_or_button, "Approve Project")
expect(page).to have_selector(:link_or_button, "Deny Project")
expect(page).to have_selector(:link_or_button, "Review Contents")
end
end
Expand Down Expand Up @@ -248,7 +245,6 @@
expect(page).not_to have_content "This project has not been saved to Mediaflux"
expect(page).not_to have_content pending_text
expect(page).to have_selector(:link_or_button, "Approve Project")
expect(page).to have_selector(:link_or_button, "Deny Project")
expect(page).to have_selector(:link_or_button, "Return to Dashboard")
end

Expand All @@ -270,7 +266,6 @@
expect(page).to have_content "This project has not been saved to Mediaflux"
expect(page).to have_content pending_text
expect(page).to have_selector(:link_or_button, "Approve Project")
expect(page).to have_selector(:link_or_button, "Deny Project")
expect(page).to have_selector(:link_or_button, "Return to Dashboard")
end
end
Expand Down

0 comments on commit 7a2f7b4

Please sign in to comment.