accept review comments https://github.com/NeuromatchAcademy/mastodon/…

…pull/44\#discussion_r1766143286 and https://github.com/NeuromatchAcademy/mastodon/pull/44\#discussion_r1766148179
NeuromatchAcademy · Sep 19, 2024 · 18e05fd · 18e05fd
1 parent 39ac978
commit 18e05fd
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/app/services/activitypub/fetch_replies_service.rb b/app/services/activitypub/fetch_replies_service.rb
@@ -25,7 +25,7 @@ def call(parent_status, collection_or_uri, allow_synchronous_requests: true, req
 
     FetchReplyWorker.push_bulk(filtered_replies) { |reply_uri| [reply_uri, { 'request_id' => request_id, 'all_replies' => @all_replies }] }
     # Store last fetched all to debounce
-    @status.update(fetched_replies_at: Time.now.utc) if fetch_all_replies?
+    @status.touch(:fetched_replies_at)
 
     @items
   end

diff --git a/db/migrate/20240918233930_add_fetched_replies_at_to_status.rb b/db/migrate/20240918233930_add_fetched_replies_at_to_status.rb
@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
 class AddFetchedRepliesAtToStatus < ActiveRecord::Migration[7.1]
-  disable_ddl_transaction!
-
   def change
     add_column :statuses, :fetched_replies_at, :datetime, null: true
   end