Browse Source

Add no merge requests message

pull/2561/head
Manny Dinssa 2 years ago
parent
commit
683a6eeb29
  1. 58
      app/views/organisations/_merge_request_list.html.erb

58
app/views/organisations/_merge_request_list.html.erb

@ -1,4 +1,7 @@
<section class="app-table-group" tabindex="0" aria-labelledby="<%= title.dasherize %>"> <section class="app-table-group" tabindex="0" aria-labelledby="<%= title.dasherize %>">
<% if @merge_requests.empty? %>
<p>No merge requests</p>
<% else %>
<%= govuk_table do |table| %> <%= govuk_table do |table| %>
<%= table.with_caption(classes: %w[govuk-!-font-size-19 govuk-!-font-weight-regular]) do |caption| %> <%= table.with_caption(classes: %w[govuk-!-font-size-19 govuk-!-font-weight-regular]) do |caption| %>
<strong><%= @merge_requests.where.not(status: 4).count %></strong> unresolved merge requests <strong><%= @merge_requests.where.not(status: 4).count %></strong> unresolved merge requests
@ -19,33 +22,34 @@
}) %> }) %>
<% end %> <% end %>
<% end %> <% end %>
<% @merge_requests.each do |merge_request| %> <% @merge_requests.each do |merge_request| %>
<%= table.with_body do |body| %> <%= table.with_body do |body| %>
<%= body.with_row do |row| %> <%= body.with_row do |row| %>
<% if merge_request.absorbing_organisation_name.blank? %> <% if merge_request.absorbing_organisation_name.blank? %>
<% row.with_cell(text: "You didn't answer this question", <% row.with_cell(text: "You didn't answer this question",
html_attributes: { html_attributes: {
scope: "row", scope: "row",
class: "app-!-colour-muted", class: "app-!-colour-muted",
}) %> }) %>
<% else %> <% else %>
<% row.with_cell(text: merge_request.absorbing_organisation_name) %> <% row.with_cell(text: merge_request.absorbing_organisation_name) %>
<% end %> <% end %>
<% merge_date = merge_request.merge_date %> <% merge_date = merge_request.merge_date %>
<% if merge_date.nil? %> <% if merge_date.nil? %>
<% row.with_cell(text: "You didn't answer this question", <% row.with_cell(text: "You didn't answer this question",
html_attributes: { html_attributes: {
scope: "row", scope: "row",
class: "app-!-colour-muted", class: "app-!-colour-muted",
}) %> }) %>
<% else %> <% else %>
<% row.with_cell(text: merge_date.strftime("%d %B %Y")) %> <% row.with_cell(text: merge_date.strftime("%d %B %Y")) %>
<% end %> <% end %>
<% row.with_cell(text: status_tag(merge_request.status)) %> <% row.with_cell(text: status_tag(merge_request.status)) %>
<% row.with_cell(html_attributes: { <% row.with_cell(html_attributes: {
scope: "row", scope: "row",
}) do %> }) do %>
<%= govuk_link_to("View details", "/merge-request/#{merge_request.id}/organisations") %> <%= govuk_link_to("View details", "/merge-request/#{merge_request.id}/organisations") %>
<% end %>
<% end %> <% end %>
<% end %> <% end %>
<% end %> <% end %>

Loading…
Cancel
Save