diff --git a/app/assets/stylesheets/application.scss b/app/assets/stylesheets/application.scss index 2b1c62859f..538ea29f3d 100644 --- a/app/assets/stylesheets/application.scss +++ b/app/assets/stylesheets/application.scss @@ -31,6 +31,7 @@ @use "pages/case_contacts"; @use "pages/court_dates"; @use "pages/emancipation"; +@use "pages/feature_flags"; @use "pages/login"; @use "pages/password"; @use "pages/patch_notes"; diff --git a/app/assets/stylesheets/pages/feature_flags.scss b/app/assets/stylesheets/pages/feature_flags.scss new file mode 100644 index 0000000000..cc0fd1fef4 --- /dev/null +++ b/app/assets/stylesheets/pages/feature_flags.scss @@ -0,0 +1,9 @@ +#feature-flags { + padding-top: 1em; + h1 { + padding-bottom: 1em; + } + .flag-name { + padding-right: 2em; + } +} \ No newline at end of file diff --git a/app/controllers/all_casa_admins/feature_flags_controller.rb b/app/controllers/all_casa_admins/feature_flags_controller.rb new file mode 100644 index 0000000000..c1d75f76ed --- /dev/null +++ b/app/controllers/all_casa_admins/feature_flags_controller.rb @@ -0,0 +1,10 @@ +class AllCasaAdmins::FeatureFlagsController < AllCasaAdminsController + def index + @feature_flags = FeatureFlag.all.order(name: "asc") + end + + def update + @feature_flag = FeatureFlag.find(params[:id]) + @feature_flag.update(enabled: !@feature_flag.enabled) + end +end diff --git a/app/javascript/all_casa_admin.js b/app/javascript/all_casa_admin.js index 6324ded939..97d90c31ef 100644 --- a/app/javascript/all_casa_admin.js +++ b/app/javascript/all_casa_admin.js @@ -1,4 +1,5 @@ require('./src/all_casa_admin/tables') require('./src/all_casa_admin/patch_notes') +require('./src/all_casa_admin/feature_flags') require('./src/session_timeout_poller.js') require('./src/display_app_metric.js') diff --git a/app/javascript/src/all_casa_admin/feature_flags.js b/app/javascript/src/all_casa_admin/feature_flags.js new file mode 100644 index 0000000000..e72d0419e9 --- /dev/null +++ b/app/javascript/src/all_casa_admin/feature_flags.js @@ -0,0 +1,30 @@ +$(function () { + const checkboxes = $('.form-check-input.toggle-switch') + + checkboxes.on('change', function (e) { + e.preventDefault() + const featureFlagId = $(this).data('feature-flag-id') + const uniqueCheckbox = $('#feature-checkbox-' + featureFlagId) + const csrfToken = $('meta[name=csrf-token]').attr('content') + const isChecked = this.checked + toggleSwitch(uniqueCheckbox, csrfToken, featureFlagId, isChecked) + }) + + function toggleSwitch (uniqueCheckbox, csrfToken, featureFlagId, checked) { + $.ajax({ + url: `/all_casa_admins/feature_flags/${featureFlagId}`, + method: 'PATCH', + contentType: 'application/json', + headers: { + 'X-CSRF-Token': csrfToken + }, + success: function () { + if (checked) { + uniqueCheckbox.prop('checked', true) + } else { + uniqueCheckbox.prop('checked', false) + } + } + }) + } +}) diff --git a/app/models/feature_flag.rb b/app/models/feature_flag.rb index cb7ed4391a..5b996ba80a 100644 --- a/app/models/feature_flag.rb +++ b/app/models/feature_flag.rb @@ -1,4 +1,5 @@ class FeatureFlag < ApplicationRecord + validates :name, presence: true, uniqueness: true end # == Schema Information diff --git a/app/views/all_casa_admins/feature_flags/index.html.erb b/app/views/all_casa_admins/feature_flags/index.html.erb new file mode 100644 index 0000000000..8e19f73c20 --- /dev/null +++ b/app/views/all_casa_admins/feature_flags/index.html.erb @@ -0,0 +1,18 @@ +
+

Feature Flags

+ + + <% @feature_flags.each do |feature_flag| %> + + + + + <% end %> + +
<%= feature_flag.name %> +
+ checked <% end %>> + +
+
+
diff --git a/app/views/layouts/_all_casa_admin_sidebar.html.erb b/app/views/layouts/_all_casa_admin_sidebar.html.erb index 32e29dca71..e029362731 100644 --- a/app/views/layouts/_all_casa_admin_sidebar.html.erb +++ b/app/views/layouts/_all_casa_admin_sidebar.html.erb @@ -22,6 +22,12 @@ Edit Profile <% end %> +
  • + <%= link_to all_casa_admins_feature_flags_path do %> + + Feature Flags + <% end %> +
  • <%= link_to destroy_all_casa_admin_session_path, id:"all-casa-log-out" do %> diff --git a/config/routes.rb b/config/routes.rb index 4bf77eb278..2f48faa51a 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -166,6 +166,8 @@ end resources :patch_notes, only: %i[create destroy index update] + + resources :feature_flags, only: %i[index update] end resources :all_casa_admins, only: [:new, :create] do diff --git a/public/assets/css/main.css b/public/assets/css/main.css index 8151641397..2086099eb5 100644 --- a/public/assets/css/main.css +++ b/public/assets/css/main.css @@ -3859,4 +3859,4 @@ h6, font-weight: 800; } .fw-900 { - font-weight: 900; } + font-weight: 900; } \ No newline at end of file diff --git a/spec/models/feature_flag_spec.rb b/spec/models/feature_flag_spec.rb index 34f8791b44..35e797420f 100644 --- a/spec/models/feature_flag_spec.rb +++ b/spec/models/feature_flag_spec.rb @@ -8,7 +8,7 @@ expect { create(:feature_flag, name: duplicate_feature_flag_name) create(:feature_flag, name: duplicate_feature_flag_name) - }.to raise_error(ActiveRecord::RecordNotUnique) + }.to raise_error(ActiveRecord::RecordInvalid) end end end diff --git a/spec/requests/all_casa_admins/feature_flags_spec.rb b/spec/requests/all_casa_admins/feature_flags_spec.rb new file mode 100644 index 0000000000..77d8ffbf19 --- /dev/null +++ b/spec/requests/all_casa_admins/feature_flags_spec.rb @@ -0,0 +1,26 @@ +require "rails_helper" + +RSpec.describe "/all_casa_admins/feature_flags", type: :request do + let(:all_casa_admin) { build(:all_casa_admin) } + let(:feature_flag) { create(:feature_flag) } + + before(:each) { sign_in all_casa_admin } + + describe "GET /index" do + it "gets all the feature flags" do + get all_casa_admins_feature_flags_path + + expect(response).to render_template(:index) + expect(response).to be_successful + end + end + + describe "PATCH /update" do + it "updates enable to false on toggle" do + patch "/all_casa_admins/feature_flags/#{feature_flag.id}" + + feature_flag.reload + expect(feature_flag.enabled).to eq(false) + end + end +end diff --git a/spec/views/all_casa_admins/feature_flags/index.html.erb_spec.rb b/spec/views/all_casa_admins/feature_flags/index.html.erb_spec.rb new file mode 100644 index 0000000000..dba6ba9085 --- /dev/null +++ b/spec/views/all_casa_admins/feature_flags/index.html.erb_spec.rb @@ -0,0 +1,20 @@ +require "rails_helper" + +RSpec.describe "patch_notes/index", type: :view do + let(:all_casa_admin) { build(:all_casa_admin) } + + before(:each) { sign_in all_casa_admin } + + describe "the feature flag list" do + it "displays the feature flags with toggles" do + feature_flag_1 = create(:feature_flag, name: "Test feature one") + feature_flag_2 = create(:feature_flag, name: "Test feature two") + assign(:feature_flags, [feature_flag_1, feature_flag_2]) + + render template: "all_casa_admins/feature_flags/index" + + expect(rendered).to include("Test feature one") + expect(rendered).to include("Test feature two") + end + end +end