Browse Source

Display banner to support

pull/2086/head
Kat 2 years ago
parent
commit
618ef9f097
  1. 9
      app/components/missing_stock_owners_banner_component.rb
  2. 5
      app/views/organisations/logs.html.erb

9
app/components/missing_stock_owners_banner_component.rb

@ -3,22 +3,21 @@ class MissingStockOwnersBannerComponent < ViewComponent::Base
attr_reader :user, :organisation attr_reader :user, :organisation
def initialize(user:) def initialize(user:, organisation: nil)
@user = user @user = user
@organisation = user.organisation @organisation = organisation || user.organisation
super super
end end
def display_banner? def display_banner?
return false if user.support?
return false if DataProtectionConfirmationBannerComponent.new(user:, organisation:).display_banner? return false if DataProtectionConfirmationBannerComponent.new(user:, organisation:).display_banner?
!organisation.holds_own_stock? && organisation.stock_owners.empty? && organisation.absorbed_organisations.empty? !organisation.holds_own_stock? && organisation.stock_owners.empty? && organisation.absorbed_organisations.empty?
end end
def header_text def header_text
if user.data_coordinator? if user.data_coordinator? || user.support?
"Your organisation does not own stock. You must #{add_stock_owner_link} before you can create logs.".html_safe "Your organisation does not own stock. You must #{add_stock_owner_link} before you can create logs.".html_safe
else else
"Your organisation does not own stock. You must add a stock owner before you can create logs.".html_safe "Your organisation does not own stock. You must add a stock owner before you can create logs.".html_safe
@ -26,7 +25,7 @@ class MissingStockOwnersBannerComponent < ViewComponent::Base
end end
def banner_text def banner_text
if user.data_coordinator? if user.data_coordinator? || user.support?
"If your organisation does own stock, #{contact_helpdesk_link} to update your details.".html_safe "If your organisation does own stock, #{contact_helpdesk_link} to update your details.".html_safe
else else
"Ask a data coordinator to add a stock owner. Find your data coordinators on the #{users_link}.</br> "Ask a data coordinator to add a stock owner. Find your data coordinators on the #{users_link}.</br>

5
app/views/organisations/logs.html.erb

@ -17,6 +17,11 @@
organisation: @organisation, organisation: @organisation,
) %> ) %>
<%= render MissingStockOwnersBannerComponent.new(
user: current_user,
organisation: @organisation,
) %>
<% if @duplicate_sets_count&.positive? %> <% if @duplicate_sets_count&.positive? %>
<%= govuk_notification_banner(title_text: "Important", text: govuk_link_to("Review logs", organisation_duplicates_path(@organisation, referrer: "duplicate_logs_banner"))) do |banner| %> <%= govuk_notification_banner(title_text: "Important", text: govuk_link_to("Review logs", organisation_duplicates_path(@organisation, referrer: "duplicate_logs_banner"))) do |banner| %>
<% banner.with_heading(text: I18n.t("notification.duplicate_sets", count: @duplicate_sets_count)) %> <% banner.with_heading(text: I18n.t("notification.duplicate_sets", count: @duplicate_sets_count)) %>

Loading…
Cancel
Save