Show consent actions on the summary card

Move follow-up, withdraw, and invalidate action links from the consent show page's action list into the consent card heading, so nurses can act directly from the patient session page without clicking through first. Action link logic is defined once in the consent helper and is shared by both the card component and the show page.

Author: murugapl

app/components/app_card_component.rb

@@ -41,13 +41,15 @@ class AppCardComponent < ViewComponent::Base
   def initialize(
     colour: nil,
     link_to: nil,
+    clickable: nil,
     feature: false,
     secondary: false,
     compact: false,
     filters: false,
     section: false
   )
     @link_to = link_to
+    @clickable = clickable.nil? ? link_to.present? : clickable
     @colour = colour
     @feature = filters || feature
     @secondary = secondary
@@ -63,7 +65,7 @@ def top_level_tag = @section ? "section" : "div"
   def card_classes
     [
       "nhsuk-card",
-      ("nhsuk-card--clickable" if @link_to.present?),
+      ("nhsuk-card--clickable" if @clickable),
       ("nhsuk-card--feature" if @feature),
       ("nhsuk-card--secondary" if @secondary),
       "app-card",

app/components/app_consent_card_component.rb

@@ -8,7 +8,7 @@ def initialize(consent, session:)
 
   def call
     render AppCardComponent.new(**card_options) do |card|
-      card.with_heading(level: 6) { heading }
+      card.with_heading(level: 6, actions:) { heading }
       render AppConsentSummaryComponent.new(
                consent,
                show_email_address: true,
@@ -28,7 +28,8 @@ def link_to
     session_patient_programme_consent_path(session, patient, programme, consent)
   end
 
-  def card_options = { link_to:, colour: "offset", compact: true }
+  def card_options =
+    { link_to:, clickable: false, colour: "offset", compact: true }
 
   def heading
     if consent.via_self_consent?
@@ -37,4 +38,8 @@ def heading
       "#{consent.name} (#{consent.who_responded})"
     end
   end
+
+  def actions
+    helpers.consent_action_links(consent, session:)
+  end
 end

app/helpers/consents_helper.rb

@@ -96,6 +96,53 @@ def consent_response_tag(consent)
     end
   end
 
+  def consent_action_links(consent, session:)
+    patient = consent.patient
+    programme = consent.programme
+
+    follow_up_link =
+      if consent.can_follow_up?
+        {
+          text: "Follow up",
+          href:
+            follow_up_session_patient_programme_consent_path(
+              session,
+              patient,
+              programme,
+              consent
+            )
+        }
+      end
+    withdraw_link =
+      if consent.can_withdraw?
+        {
+          text: "Withdraw consent",
+          href:
+            withdraw_session_patient_programme_consent_path(
+              session,
+              patient,
+              programme,
+              consent
+            )
+        }
+      end
+    invalidate_link =
+      if consent.can_invalidate?
+        {
+          text: "Mark as invalid",
+          href:
+            invalidate_session_patient_programme_consent_path(
+              session,
+              patient,
+              programme,
+              consent
+            )
+        }
+      end
+
+    [follow_up_link, withdraw_link, invalidate_link].compact
+  end
+
   def consent_parent_name(consentable)
     consentable.parent_full_name.presence ||
       (consentable.respond_to?(:parent) ? consentable.parent&.full_name : nil)

app/views/patient_sessions/consents/show.html.erb

@@ -5,25 +5,8 @@
 <%= h1 "Consent response from #{@consent.name}" %>
 
 <%= render AppActionListComponent.new do |action_list| %>
-  <% if @consent.can_follow_up? %>
-    <% action_list.with_item(
-         text: "Follow up",
-         href: follow_up_session_patient_programme_consent_path,
-       ) %>
-  <% end %>
-
-  <% if @consent.can_withdraw? %>
-    <% action_list.with_item(
-         text: "Withdraw consent",
-         href: withdraw_session_patient_programme_consent_path,
-       ) %>
-  <% end %>
-
-  <% if @consent.can_invalidate? %>
-    <% action_list.with_item(
-         text: "Mark as invalid",
-         href: invalidate_session_patient_programme_consent_path,
-       ) %>
+  <% consent_action_links(@consent, session: @session).each do |action| %>
+    <% action_list.with_item(**action) %>
   <% end %>
 <% end %>
 

spec/components/app_consent_card_component_spec.rb

@@ -39,6 +39,50 @@
   it { should have_content("Response") }
   it { should have_content("Consent given") }
 
+  describe "actions" do
+    context "when consent is given" do
+      it { should have_link("Withdraw consent") }
+      it { should have_link("Mark as invalid") }
+      it { should_not have_link("Follow up") }
+    end
+
+    context "when follow-up is requested" do
+      let(:consent) do
+        create(
+          :consent,
+          :follow_up_requested,
+          patient:,
+          parent:,
+          programme:,
+          team:,
+          submitted_at: Time.zone.local(2024, 1, 1)
+        )
+      end
+
+      it { should have_link("Follow up") }
+      it { should have_link("Mark as invalid") }
+      it { should_not have_link("Withdraw consent") }
+    end
+
+    context "when consent is invalidated" do
+      let(:consent) do
+        create(
+          :consent,
+          :invalidated,
+          patient:,
+          parent:,
+          programme:,
+          team:,
+          submitted_at: Time.zone.local(2024, 1, 1)
+        )
+      end
+
+      it { should_not have_link("Follow up") }
+      it { should_not have_link("Withdraw consent") }
+      it { should_not have_link("Mark as invalid") }
+    end
+  end
+
   context "with the flu programme" do
     let(:programme) { Programme.flu }
     let(:consent) { create(:consent, programme:, vaccine_methods: %w[nasal]) }