standard-editions/ID/featurings

Author: patrickpatrickpatrick

app/controllers/admin/topical_event_featurings_controller.rb

@@ -17,6 +17,13 @@ def index
 
     @topical_event_featurings = @topical_event.topical_event_featurings
     @featurable_offsite_links = @topical_event.offsite_links
+    @featurable_editions = if @topical_event.is_a?(StandardEdition)
+                            @topical_event.topical_event_documents
+                                          .page(params[:page])
+                                          .per(Admin::EditionFilter::GOVUK_DESIGN_SYSTEM_PER_PAGE)
+                           else
+                            @topical_event.featurable_editions(@tagged_editions)
+                           end
   end
 
   def new
@@ -89,6 +96,8 @@ def editions_to_show
     else
       editions = if @topical_event.respond_to?(:topical_event_documents)
                     @topical_event.topical_event_documents
+                                  .page(params[:page])
+                                  .per(Admin::EditionFilter::GOVUK_DESIGN_SYSTEM_PER_PAGE)
                   else
                     @topical_event.editions.published
                                            .with_translations

app/models/concerns/edition/offsite_links.rb

@@ -0,0 +1,18 @@
+module Edition::OffsiteLinks
+  extend ActiveSupport::Concern
+
+  class Trait < Edition::Traits::Trait
+    def process_associations_before_save(edition)
+      @edition.edition_offsite_links.each do |association|
+        edition.edition_offsite_links.build(association.attributes.except("id"))
+      end
+    end
+  end
+
+  included do
+    has_many :edition_offsite_links, foreign_key: :edition_id, dependent: :destroy
+    has_many :offsite_links, through: :edition_offsite_links, validate: false
+
+    add_trait Trait
+  end
+end

app/models/configurable_document_types/topical_event.json

@@ -54,7 +54,7 @@
   "associations": [
     {
       "key": "organisations",
-      "key": "offsite_links",
+      "key": "offsite_links"
     }
   ],
   "settings": {

app/models/edition_offsite_link.rb

@@ -0,0 +1,4 @@
+class EditionOffsiteLink < ApplicationRecord
+  belongs_to :edition
+  belongs_to :offsite_link, inverse_of: :edition_offsite_links
+end

app/models/standard_edition.rb

@@ -8,13 +8,17 @@ class StandardEdition < Edition
   include Edition::WorldLocations
   include Edition::Organisations
   include Edition::WorldwideOrganisations
+  include Edition::OffsiteLinks
   include HasBlockContent
   include StandardEdition::DefaultLeadImage
 
   validates :configurable_document_type, presence: true, inclusion: { in: -> { ConfigurableDocumentType.all_keys } }
-
   scope :with_news_article_document_type, -> { where(configurable_document_type: ConfigurableDocumentType.where_group("news_article").map(&:key)) }
 
+  def name
+    title
+  end
+
   def format_name
     type_instance.label.downcase
   end

app/views/admin/topical_event_featurings/index.html.erb

@@ -42,7 +42,7 @@
       content: render("admin/shared/featurable_editions",
         filter: @filter,
         paginator: @tagged_editions,
-        featurable_editions: @topical_event.featurable_editions(@tagged_editions),
+        featurable_editions: @featurable_editions,
         filter_by: [:title, :type, :author, :organisation],
         anchor: "#documents_tab",
         filter_action: admin_topical_event_topical_event_featurings_url(@topical_event),
@@ -54,7 +54,7 @@
       label: "Non-GOV.UK government links",
       content: render("admin/feature_lists/featureable_offsite_links",
         model: @topical_event,
-        featurable_offsite_links: @topical_event.featurable_offsite_links,
+        featurable_offsite_links: @featurable_offsite_links,
         featuring_path: [:new, :admin, @topical_event, :topical_event_featuring],
       ),
     },

config/routes.rb

@@ -315,7 +315,11 @@ def redirect(path, options = { prefix: Whitehall.router_prefix })
           get :reorder, on: :collection
           put :order, on: :collection
           get :confirm_destroy, on: :member
-        end        
+        end
+
+        resources :offsite_links do
+          get :confirm_destroy, on: :member
+        end
 
         resources :translations, controller: "standard_edition_translations", except: %i[index show create] do
           get :confirm_destroy, on: :member

db/migrate/20260109164506_create_edition_offsite_links.rb

@@ -0,0 +1,10 @@
+class CreateEditionOffsiteLinks < ActiveRecord::Migration[8.0]
+  def change
+    create_table :edition_offsite_links do |t|
+      t.timestamps
+    end
+
+    add_reference :edition_offsite_links, :edition
+    add_reference :edition_offsite_links, :offsite_link
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[8.0].define(version: 2026_01_07_195134) do
+ActiveRecord::Schema[8.0].define(version: 2026_01_09_164506) do
   create_table "assets", charset: "utf8mb4", collation: "utf8mb4_unicode_ci", force: :cascade do |t|
     t.string "asset_manager_id", null: false
     t.string "variant", null: false
@@ -275,6 +275,15 @@
     t.index ["edition_id"], name: "index_edition_links_on_edition_id"
   end
 
+  create_table "edition_offsite_links", charset: "utf8mb4", collation: "utf8mb4_0900_ai_ci", force: :cascade do |t|
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.bigint "edition_id"
+    t.bigint "offsite_link_id"
+    t.index ["edition_id"], name: "index_edition_offsite_links_on_edition_id"
+    t.index ["offsite_link_id"], name: "index_edition_offsite_links_on_offsite_link_id"
+  end
+
   create_table "edition_organisations", id: :integer, charset: "utf8mb4", collation: "utf8mb4_unicode_ci", force: :cascade do |t|
     t.integer "edition_id"
     t.integer "organisation_id"