Expand Featureable to all Tools (#4094)

This adds `featured_status` and `featured_at` to tables for tools that
didn't have it:

- logos
- podcasts
- episodes
- journals
- videos

app/lib/feature_flag.rb

@@ -1,3 +1,4 @@
+# TODO: delete this file
 # Start your Rails server with this ENV var:
 #     TWO_POINT_OH_YEAH=true bundle exec rails server
 #

app/models/book.rb

@@ -1,6 +1,5 @@
 class Book < ApplicationRecord
   include MultiPageTool
-  include Featureable
 
   def ask_for_donation?
     downloads_available?

app/models/concerns/featureable.rb

@@ -1,11 +1,10 @@
 module Featureable
   extend ActiveSupport::Concern
 
-  # TEMP: re-enable and expand coverage to include Journal, Issue, et al
-  # included do
-  #   scope :featured, -> { where.not(featured_at: nil) }
-  #   before_save :update_featured_at
-  # end
+  included do
+    scope :featured, -> { where.not(featured_at: nil) }
+    before_save :update_featured_at
+  end
 
   module ClassMethods
     def for_index fallback_sort: { title: :asc }, fallback_locale: 'en'

app/models/episode.rb

@@ -1,5 +1,6 @@
 class Episode < ApplicationRecord
   include Name
+  include Featureable
   include Publishable
 
   belongs_to :podcast

app/models/issue.rb

@@ -1,6 +1,5 @@
 class Issue < ApplicationRecord
   include MultiPageTool
-  include Featureable
 
   belongs_to :journal
 

app/models/logo.rb

@@ -1,5 +1,7 @@
 class Logo < ApplicationRecord
   include Tool
+  include Featureable
+
   has_one_attached :image_jpg, dependent: :destroy
   has_one_attached :image_png, dependent: :destroy
   has_one_attached :image_pdf, dependent: :destroy

app/models/podcast.rb

@@ -1,5 +1,6 @@
 class Podcast < ApplicationRecord
   include Name
+  include Featureable
 
   has_many :episodes, dependent: :destroy
 

app/models/video.rb

@@ -1,5 +1,6 @@
 class Video < ApplicationRecord
   include Tool
+  include Featureable
 
   has_one_attached :image_poster_frame
 

db/migrate/20241125231604_add_featured_to_all_tools.rb

@@ -0,0 +1,33 @@
+class AddFeaturedToAllTools < ActiveRecord::Migration[7.2]
+  def change
+    # logos
+    change_table :logos, bulk: true do |t|
+      t.boolean  :featured_status, default: false, null: false
+      t.datetime :featured_at,     precision: nil
+    end
+
+    # podcasts
+    change_table :podcasts, bulk: true do |t|
+      t.boolean  :featured_status, default: false, null: false
+      t.datetime :featured_at, precision: nil
+    end
+
+    # episodes
+    change_table :episodes, bulk: true do |t|
+      t.boolean  :featured_status, default: false, null: false
+      t.datetime :featured_at, precision: nil
+    end
+
+    # journals
+    change_table :journals, bulk: true do |t|
+      t.boolean  :featured_status, default: false, null: false
+      t.datetime :featured_at, precision: nil
+    end
+
+    # videos
+    change_table :videos, bulk: true do |t|
+      t.boolean  :featured_status, default: false, null: false
+      t.datetime :featured_at, precision: nil
+    end
+  end
+end