Crop job admin

app/controllers/admin/crop_job_templates_controller.rb

@@ -10,7 +10,7 @@ module Admin
     before_action :set_template, only: %i(edit update destroy)
 
     def index
-      @templates = CropJobTemplate.where(crop_id: crop_ids).includes(:crop, :steps).order(:name)
+      @templates = CropJobTemplate.where(crop_id: crop_ids).includes(:crop, :creator, :steps).order(:name)
     end
 
     def new
@@ -24,7 +24,7 @@ module Admin
     end
 
     def create
-      @template = CropJobTemplate.new(template_params)
+      @template = CropJobTemplate.new(template_params.merge(creator: current_member))
       return render_with_another_step(:new) if add_step?
 
       if @template.save

app/controllers/admin/job_templates_controller.rb

@@ -9,7 +9,7 @@ module Admin
 
     def index
       @status = params[:status] if CropJobTemplate.statuses.key?(params[:status])
-      @templates = CropJobTemplate.joins(:crop).includes(:steps, crop: :parent).order('crops.name', :name)
+      @templates = CropJobTemplate.joins(:crop).includes(:creator, :steps, crop: :parent).order('crops.name', :name)
       @templates = @templates.where(status: @status) if @status
       @templates = @templates.paginate(page: params[:page])
     end

app/models/crop_job_template.rb

@@ -2,9 +2,11 @@
 
 # A crop's suggested jobs, offered to gardeners who plant it. It belongs to the
 # crop it is written for and applies to that crop's varieties too. Only
-# published templates are offered.
+# published templates are offered. Its creator is the member who wrote it;
+# templates loaded from the seed files have none.
 class CropJobTemplate < ApplicationRecord
   belongs_to :crop
+  belongs_to :creator, class_name: 'Member', optional: true, inverse_of: :created_crop_job_templates
   has_many :steps, -> { order(:position, :id) }, class_name: 'CropJobStep', dependent: :destroy, inverse_of: :crop_job_template
 
   # Which months a season is depends on the gardener's hemisphere, so it is

app/models/member.rb

@@ -81,6 +81,8 @@ class Member < ApplicationRecord
                            inverse_of: :creator
   has_many :created_alternate_names, class_name: 'AlternateName', foreign_key: 'creator_id', inverse_of: :creator
   has_many :created_scientific_names, class_name: 'ScientificName', foreign_key: 'creator_id', inverse_of: :creator
+  has_many :created_crop_job_templates, class_name: 'CropJobTemplate', foreign_key: 'creator_id', dependent: :nullify,
+                                        inverse_of: :creator
 
   #
   # Scopes

app/views/admin/crop_job_templates/index.html.haml

@@ -22,6 +22,7 @@
         %th Name
         %th Applies to
         %th Season
+        %th Written by
         %th Steps
         %th Status
         %th
@@ -31,6 +32,7 @@
           %td= link_to template.name, edit_admin_crop_crop_job_template_path(@crop, template)
           %td= template.crop == @crop ? t('crop_job_templates.options.applies_to_all', crop: @crop.name) : t('crop_job_templates.options.applies_to_one', crop: template.crop.name)
           %td= crop_job_option(:season, template.season)
+          %td= template.creator ? link_to(template.creator, template.creator) : 'Seed data'
           %td= template.steps.size
           %td
             %span.badge{ class: template.published? ? 'text-bg-success' : 'text-bg-secondary' }= crop_job_option(:status, template.status)

app/views/admin/job_templates/index.html.haml

@@ -26,6 +26,7 @@
         %th Crop
         %th Name
         %th Season
+        %th Written by
         %th Steps
         %th Status
     %tbody
@@ -35,6 +36,7 @@
           %td= link_to template.crop.name, admin_crop_crop_job_templates_path(crop)
           %td= link_to template.name, edit_admin_crop_crop_job_template_path(crop, template)
           %td= crop_job_option(:season, template.season)
+          %td= template.creator ? link_to(template.creator, template.creator) : 'Seed data'
           %td= template.steps.size
           %td
             - badge = template.published? ? 'text-bg-success' : 'text-bg-secondary'

db/schema.rb

@@ -545,12 +545,12 @@ ActiveRecord::Schema[8.1].define(version: 2026_10_24_100000) do
 
   create_table "crop_job_templates", force: :cascade do |t|
     t.datetime "created_at", null: false
+    t.bigint "creator_id"
     t.bigint "crop_id", null: false
     t.string "name", null: false
     t.string "season", default: "any", null: false
     t.string "status", default: "draft", null: false
     t.datetime "updated_at", null: false
-    t.bigint "creator_id"
     t.index ["creator_id"], name: "index_crop_job_templates_on_creator_id"
     t.index ["crop_id"], name: "index_crop_job_templates_on_crop_id"
   end

spec/requests/admin/crop_job_templates_controller_spec.rb

@@ -57,6 +57,10 @@ describe Admin::CropJobTemplatesController do
         expect(crop.crop_job_templates.first).to be_draft
       end
 
+      it 'records who wrote it' do
+        expect(crop.crop_job_templates.first.creator).to eq wrangler
+      end
+
       it 'goes back to the list' do
         expect(response).to redirect_to(admin_crop_crop_job_templates_path(crop))
       end

spec/requests/admin/job_templates_controller_spec.rb

@@ -5,9 +5,10 @@ require 'rails_helper'
 describe Admin::JobTemplatesController do
   let(:tomato) { create(:crop, name: 'tomato') }
   let(:cherry) { create(:crop, name: 'cherry tomato', parent: tomato) }
+  let(:author) { create(:member, login_name: 'template_author') }
 
   before do
-    create(:crop_job_template, crop: tomato, name: 'Growing tomatoes', status: 'published')
+    create(:crop_job_template, crop: tomato, name: 'Growing tomatoes', status: 'published', creator: author)
     create(:crop_job_template, crop: cherry, name: 'Growing cherry tomatoes')
     create(:crop_job_template, name: 'Growing something else')
   end
@@ -23,6 +24,10 @@ describe Admin::JobTemplatesController do
           expect(response.body).to include('Growing tomatoes', 'Growing cherry tomatoes', 'Growing something else')
         end
 
+        it 'shows who wrote a template' do
+          expect(response.body).to include('template_author')
+        end
+
         it 'links a variety\'s template to its parent crop\'s templates' do
           expect(response.body).to include(edit_admin_crop_crop_job_template_path(tomato, cherry.crop_job_templates.first))
         end