Make the deadline field more agnostic

- optional no longer mandatory

Author: hezzimo

app/models/request.rb

@@ -3,7 +3,7 @@ class Request < ApplicationRecord
   has_many :users, through: :collaborations
   has_one :response
 
-  validates :source_id, :source_app, :requester_name, :requester_email, :status, :current_content, :deadline, presence: true
+  validates :source_id, :source_app, :requester_name, :requester_email, :status, :current_content, presence: true
   scope :for_source, ->(source_id) { where(source_id: source_id) }
   scope :most_recent_first, -> { order(created_at: :desc) }
   scope :most_recent_for_source, ->(source_id) { for_source(source_id).most_recent_first.first }

db/migrate/20260107162918_create_requests.rb

@@ -12,7 +12,7 @@ def change
       t.string :status, null: false, default: 'in_progress'
       t.text :previous_content
       t.text :current_content, null: false
-      t.datetime :deadline, null: false
+      t.datetime :deadline
 
       t.timestamps
     end

db/schema.rb

@@ -33,7 +33,7 @@
     t.string "status", default: "in_progress", null: false
     t.text "previous_content"
     t.text "current_content", null: false
-    t.datetime "deadline", null: false
+    t.datetime "deadline"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
     t.string "source_app", null: false

spec/models/request_spec.rb

@@ -7,11 +7,11 @@
     expect(record).not_to be_valid
   end
 
-  it "includes errors for each missing attribute" do
+  it "includes errors for each missing mandatory attribute" do
     record = described_class.new
     record.valid?
 
-    expect(record.errors.attribute_names).to include(:source_id, :source_app, :requester_name, :requester_email, :current_content, :deadline)
+    expect(record.errors.attribute_names).to include(:source_id, :source_app, :requester_name, :requester_email, :current_content)
   end
 
   it "is valid when all required attributes are set" do