Whether feature flag is enabled is per user

openaustralia · Aug 16, 2024 · 195724f · 195724f
1 parent 71017c3
commit 195724f
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/app/controllers/api_keys_controller.rb b/app/controllers/api_keys_controller.rb
@@ -16,7 +16,7 @@ def index
   sig { void }
   def create
     user = T.must(current_user)
-    redirect_url = if Flipper.enabled?(:api_keys_in_profile)
+    redirect_url = if Flipper.enabled?(:api_keys_in_profile, current_user)
                      api_keys_url
                    else
                      api_howto_url

diff --git a/app/views/application/_profile_menu.html.erb b/app/views/application/_profile_menu.html.erb
@@ -2,7 +2,7 @@
   <li><div class="pl-4"><%= pa_link_to_unless_current "Your alerts", alerts_path %></div></li>
   <li><div class="pl-4"><%= pa_link_to_unless_current "Create new alert", new_alert_path %></div></li>
   <li><div class="pl-4"><%= pa_link_to_unless_current "Your comments", personal_comments_path %></div></li>
-  <% if Flipper.enabled?(:api_keys_in_profile) %>
+  <% if Flipper.enabled?(:api_keys_in_profile, current_user) %>
     <li><div class="pl-4"><%= pa_link_to_unless_current "Your API keys", api_keys_path %></div></li>
   <% end %>
   <li><div class="pl-4"><%= pa_link_to_unless_current "Edit account", edit_user_registration_path %></div></li>

diff --git a/app/views/documentation/api_howto.html.erb b/app/views/documentation/api_howto.html.erb
@@ -33,7 +33,7 @@
           You need an API key to try out the examples below and use the API for yourself.
         </p>
         <div class="mt-8 mb-4">
-          <% if Flipper.enabled?(:api_keys_in_profile) %>
+          <% if Flipper.enabled?(:api_keys_in_profile, current_user) %>
             <%= render ButtonComponent.new(tag: :a, size: "xl", type: :primary, href: confirm_api_keys_path).with_content("Create API key") %>
           <% else %>
             <%= pa_button_to "Create API key", api_keys_path, method: :post %>