Comments

Author: naofumi

app/models/feature_toggles.rb

@@ -1,5 +1,6 @@
+# Central location to store FeatureToggles
 class FeatureToggles
   class << self
-    def reset_data = true
+    def reset_data? = true
   end
 end

app/views/contexts/show.json.jbuilder

@@ -9,7 +9,9 @@ json.current_user do
   end
 end
 
-json.feature_flags reset_data_feature: FeatureToggles.reset_data
+# We often see feature toggle definitions scattered throughout the codebase.
+# They should be put in a central location – here we use FeatureToggles.
+json.feature_flags reset_data_feature: FeatureToggles.reset_data?
 json.theme "light-mode"
 json.locale I18n.locale
 json.environment Rails.env

app/views/posts/index.json.jbuilder

@@ -15,7 +15,7 @@ json.posts do
     end
     json.highlighted Permissions::Post.can_edit?(current_user, post)
     # We send permissions to the server so that
-    # the client doesn't have to contain authorization logic.
+    # the client doesn't have to contain any authorization logic.
     json.can_edit_post Permissions::Post.can_edit?(current_user, post)
     json.created_at post.created_at.iso8601
   end
@@ -26,7 +26,7 @@ json.pagination do
   json.prev_page @posts.prev_page
 end
 # We send permissions to the server so that
-# the client doesn't have to contain authorization logic.
+# the client doesn't have to contain any authorization logic.
 json.permissions do
   json.can_create_post Permissions::Post.can_create?(current_user)
 end