Merge pull request #5764 from nhsuk/mike/remove-reports-card-in-national-reporting

Remove reports card from national reporting dashboard

Author: mikejamesthompson

app/controllers/dashboard_controller.rb

@@ -23,18 +23,13 @@ def set_primary_items
             title: I18n.t("imports.index.title"),
             path: imports_path,
             description: I18n.t("imports.index.description_upload_only"),
-            width: "one-third"
+            width: "one-half"
           },
           {
             title: I18n.t("patients.index.title"),
             path: patients_path,
             description: I18n.t("patients.index.description_upload_only"),
-            width: "one-third"
-          },
-          {
-            title: I18n.t("reports.index.title"),
-            description: I18n.t("reports.index.description_upload_only"),
-            width: "one-third"
+            width: "one-half"
           }
         ]
       else
@@ -70,7 +65,7 @@ def set_primary_items
   def set_secondary_items
     @secondary_items = []
 
-    unless current_team.has_upload_only_access?
+    if current_team.has_poc_only_access?
       @secondary_items << {
         title: I18n.t("school_moves.index.title"),
         path: school_moves_path,

config/locales/en.yml

@@ -734,7 +734,6 @@ en:
     index:
       title: Reports
       description: View the progress of vaccination programmes and download vaccination data
-      description_upload_only: View statistics on vaccinations given by vaccine, gender and year group
   school_moves:
     index:
       title: School moves

spec/features/upload_only_team_spec.rb

@@ -7,7 +7,6 @@
     then_i_should_see_the_upload_only_cards
     and_i_should_see_the_import_records_card
     and_i_should_see_the_children_card
-    and_i_should_see_the_reports_card
     and_i_should_see_the_national_reporting_service_name
   end
 
@@ -83,7 +82,7 @@ def when_i_visit_the_dashboard
 
   def then_i_should_see_the_upload_only_cards
     cards = page.all(".nhsuk-card-group__item")
-    expect(cards.count).to eq(4)
+    expect(cards.count).to eq(3)
   end
 
   def and_i_should_see_the_import_records_card
@@ -105,17 +104,6 @@ def and_i_should_see_the_children_card
     expect(card).to have_link("Children", href: patients_path)
   end
 
-  def and_i_should_see_the_reports_card
-    cards = page.all(".nhsuk-card-group__item")
-    card = cards[2]
-
-    expect(card).to have_css("h2", text: "Reports")
-    expect(card).not_to have_link("Reports")
-
-    # Card should be disabled
-    expect(card).to have_css(".app-card--disabled")
-  end
-
   def then_i_should_see_only_import_children_and_team_navigation_items
     navigation_items = page.all(".nhsuk-header__navigation-item")
     expect(navigation_items.count).to eq(2)