DEV: Standardize ignored_columns removal comments (#9771)

DEV: Standardize ignored_columns removal comments (#9771)

diff --git a/app/models/category.rb b/app/models/category.rb
index ddab403..63db814 100644
--- a/app/models/category.rb
+++ b/app/models/category.rb
@@ -5,6 +5,7 @@ class Category < ActiveRecord::Base
     'none'
   ]
 
+  # TODO(2020-11-18): remove
   self.ignored_columns = %w{
     suppress_from_latest
   }
diff --git a/app/models/group.rb b/app/models/group.rb
index 1f93c99..5bc41d7 100644
--- a/app/models/group.rb
+++ b/app/models/group.rb
@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 class Group < ActiveRecord::Base
+  # TODO(2021-04-22): remove
   self.ignored_columns = %w{
     automatic_membership_retroactive
   }
diff --git a/app/models/post.rb b/app/models/post.rb
index c2f80c8..89dbbcf 100644
--- a/app/models/post.rb
+++ b/app/models/post.rb
@@ -10,7 +10,7 @@ class Post < ActiveRecord::Base
   include HasCustomFields
   include LimitedEdit
 
-  # remove line Jan 2021
+  # TODO(2021-01-04): remove
   self.ignored_columns = ["avg_time"]
 
   cattr_accessor :plugin_permitted_create_params
diff --git a/app/models/post_reply.rb b/app/models/post_reply.rb
index 9cc9be2..fc8eee4 100644
--- a/app/models/post_reply.rb
+++ b/app/models/post_reply.rb
@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 class PostReply < ActiveRecord::Base
+  # TODO(2020-01-17): remove
   self.ignored_columns = %w{
     reply_id
   }
diff --git a/app/models/topic.rb b/app/models/topic.rb
index 371f32d..cdbd6ee 100644
--- a/app/models/topic.rb
+++ b/app/models/topic.rb
@@ -10,7 +10,7 @@ class Topic < ActiveRecord::Base
   include LimitedEdit
   extend Forwardable
 
-  # remove line Jan 2021
+  # TODO(2021-01-04): remove
   self.ignored_columns = ["avg_time"]
 
   def_delegator :featured_users, :user_ids, :featured_user_ids
diff --git a/app/models/user_email.rb b/app/models/user_email.rb
index e4bfe50..7ce64cb 100644
--- a/app/models/user_email.rb
+++ b/app/models/user_email.rb
@@ -1,6 +1,9 @@
 # frozen_string_literal: true
 
 class UserEmail < ActiveRecord::Base
+  # TODO(2020-04-24): remove
+  self.ignored_columns = ['canonical_email']
+
   belongs_to :user
 
   attr_accessor :skip_validate_email
@@ -17,8 +20,6 @@ class UserEmail < ActiveRecord::Base
 
   scope :secondary, -> { where(primary: false) }
 
-  self.ignored_columns = ['canonical_email']
-
   private
 
   def strip_downcase_email

GitHub sha: 1059557c

1 Like

This commit appears in #9771 which was approved by eviltrout. It was merged by riking.