Allow Card#last_updated_at to be set

This is useful when doing an import from another system. I'm currently
working on a script to import our Github issues into Fizzy.

This is discussed in
#2056 (comment)

Author: jankeesvw

app/controllers/cards_controller.rb

@@ -61,6 +61,6 @@ def ensure_permission_to_administer_card
     end
 
     def card_params
-      params.expect(card: [ :status, :title, :description, :image, :created_at, tag_ids: [] ])
+      params.expect(card: [ :status, :title, :description, :image, :created_at, :last_active_at, tag_ids: [] ])
     end
 end

app/models/card/eventable.rb

@@ -4,15 +4,15 @@ module Card::Eventable
   include ::Eventable
 
   included do
-    before_create { self.last_active_at = Time.current }
+    before_create { self.last_active_at ||= created_at }
 
     after_save :track_title_change, if: :saved_change_to_title?
   end
 
   def event_was_created(event)
     transaction do
       create_system_comment_for(event)
-      touch_last_active_at
+      touch_last_active_at unless was_just_published?
     end
   end
 

docs/API.md

@@ -495,6 +495,7 @@ Creates a new card in a board.
 | `image` | file | No | Header image for the card |
 | `tag_ids` | array | No | Array of tag IDs to apply to the card |
 | `created_at` | datetime | No | Override creation timestamp (ISO 8601 format) |
+| `last_active_at` | datetime | No | Override last activity timestamp (ISO 8601 format) |
 
 __Request:__
 
@@ -522,6 +523,7 @@ Updates a card.
 | `status` | string | No | Card status: `drafted`, `published` |
 | `image` | file | No | Header image for the card |
 | `tag_ids` | array | No | Array of tag IDs to apply to the card |
+| `last_active_at` | datetime | No | Override last activity timestamp (ISO 8601 format) |
 
 __Request:__
 

test/controllers/cards_controller_test.rb

@@ -173,6 +173,22 @@ class CardsControllerTest < ActionDispatch::IntegrationTest
     assert_equal custom_time, Card.last.created_at
   end
 
+  test "create as JSON with custom last_active_at" do
+    created_time = Time.utc(2024, 1, 15, 10, 30, 0)
+    last_active_time = Time.utc(2024, 6, 1, 12, 0, 0)
+
+    assert_difference -> { Card.count }, +1 do
+      post board_cards_path(boards(:writebook)),
+        params: { card: { title: "Card with activity", created_at: created_time, last_active_at: last_active_time } },
+        as: :json
+    end
+
+    assert_response :created
+    card = Card.last
+    assert_equal created_time, card.created_at
+    assert_equal last_active_time, card.last_active_at
+  end
+
   test "update as JSON" do
     card = cards(:logo)
     put card_path(card, format: :json), params: { card: { title: "Update test" } }