Merge pull request #3553 from nilbus/reduce-db-lookups

Only check $flipper feature flags for page requests
exercism · Jun 5, 2017 · 0c37457 · 0c37457
2 parents 5a59302 + e2188c3
commit 0c37457
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/app/routes/profile.rb b/app/routes/profile.rb
@@ -2,7 +2,7 @@ module ExercismWeb
   module Routes
     class Profile < Core
       before do
-        if $flipper[:advertise_cli_update].enabled?(current_user) && page_request?
+        if page_request? && $flipper[:advertise_cli_update].enabled?(current_user)
           client_version = ClientVersion.new(user: current_user)
           flash.now[:notice] ||= client_version.notice_when_client_outdated
         end