Rename CohortsController

To Programmes::CohortsController, to better structure the controllers in
relation to how they're shown on the frontend.

Author: thomasleese

app/components/app_programme_navigation_component.rb

@@ -18,7 +18,7 @@ def call
 
       nav.with_item(
         href: programme_cohorts_path(programme),
-        text: I18n.t("cohorts.index.title"),
+        text: I18n.t("programmes.cohorts.index.title"),
         selected: active == :cohorts
       )
 

app/controllers/cohorts_controller.rb …rollers/programmes/cohorts_controller.rbapp/controllers/cohorts_controller.rb renamed to app/controllers/programmes/cohorts_controller.rb app/controllers/cohorts_controller.rb renamed to app/controllers/programmes/cohorts_controller.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-class CohortsController < ApplicationController
+class Programmes::CohortsController < ApplicationController
   include Pagy::Backend
 
   before_action :set_programme

app/views/cohorts/index.html.erb …/views/programmes/cohorts/index.html.erbapp/views/cohorts/index.html.erb renamed to app/views/programmes/cohorts/index.html.erb app/views/cohorts/index.html.erb renamed to app/views/programmes/cohorts/index.html.erb

@@ -1,4 +1,4 @@
-<%= content_for :page_title, "#{@programme.name} – #{t("cohorts.index.title")}" %>
+<%= content_for :page_title, "#{@programme.name} – #{t(".title")}" %>
 
 <% content_for :before_main do %>
   <%= render AppBreadcrumbComponent.new(items: [

app/views/cohorts/show.html.erb …p/views/programmes/cohorts/show.html.erbapp/views/cohorts/show.html.erb renamed to app/views/programmes/cohorts/show.html.erb app/views/cohorts/show.html.erb renamed to app/views/programmes/cohorts/show.html.erb

@@ -3,7 +3,7 @@
                                           { text: t("dashboard.index.title"), href: dashboard_path },
                                           { text: t("programmes.index.title"), href: programmes_path },
                                           { text: @programme.name, href: programme_path(@programme) },
-                                          { text: t("cohorts.index.title"), href: programme_cohorts_path(@programme) },
+                                          { text: t("programmes.cohorts.index.title"), href: programme_cohorts_path(@programme) },
                                         ]) %>
 <% end %>
 

config/locales/en.yml

@@ -490,9 +490,6 @@ en:
     zero: No children
     one: 1 child
     other: "%{count} children"
-  cohorts:
-    index:
-      title: Cohorts
   consent_forms:
     index:
       title: Unmatched consent responses
@@ -584,6 +581,9 @@ en:
     index:
       title: Children
   programmes:
+    cohorts:
+      index:
+        title: Cohorts
     index:
       title: Programmes
     sessions:

config/routes.rb

@@ -179,7 +179,7 @@
       get "consent-form", action: "consent_form"
     end
 
-    resources :cohorts, only: %i[index show]
+    resources :cohorts, only: %i[index show], controller: "programmes/cohorts"
 
     resources :vaccination_records,
               path: "vaccination-records",