Remove FlipFlip gem

https://mitlibraries.atlassian.net/browse/USE-77

Author: JPrevost

.env.test

@@ -1,5 +1,5 @@
 ALMA_OPENURL=https://na06.alma.exlibrisgroup.com/view/uresolver/01MIT_INST/openurl?
-GDT=false
+FEATURE_GEODATA=false
 MIT_PRIMO_URL=https://mit.primo.exlibrisgroup.com
 PRIMO_API_KEY=FAKE_PRIMO_API_KEY
 PRIMO_API_URL=https://api-na.hosted.exlibrisgroup.com/primo/v1
@@ -12,4 +12,4 @@ TACOS_URL=http://FAKE_TACOS_HOST/graphql
 TACOS_SOURCE=FAKE_TACOS_SOURCE
 TIMDEX_GRAPHQL=https://FAKE_TIMDEX_HOST/graphql
 TIMDEX_HOST=FAKE_TIMDEX_HOST
-TIMDEX_INDEX=FAKE_TIMDEX_INDEX
+TIMDEX_INDEX=FAKE_TIMDEX_INDEX

Gemfile

@@ -52,5 +52,3 @@ group :test do
   gem 'vcr'
   gem 'webmock'
 end
-
-gem 'flipflop', '~> 2.7'

Gemfile.lock

@@ -139,9 +139,6 @@ GEM
       ffi (>= 1.15.5)
       rake
     fiber-storage (1.0.1)
-    flipflop (2.8.0)
-      activesupport (>= 4.0)
-      terminal-table (>= 1.8)
     globalid (1.3.0)
       activesupport (>= 6.1)
     graphql (2.5.14)
@@ -359,8 +356,6 @@ GEM
     stimulus-rails (1.3.4)
       railties (>= 6.0.0)
     stringio (3.1.7)
-    terminal-table (4.0.0)
-      unicode-display_width (>= 1.1.1, < 4)
     thor (1.4.0)
     tilt (2.6.0)
     timeout (0.4.3)
@@ -415,7 +410,6 @@ DEPENDENCIES
   climate_control
   debug
   dotenv-rails
-  flipflop (~> 2.7)
   graphql
   graphql-client
   http

README.md

@@ -63,7 +63,7 @@ requires testing against an index that contains geospatial records.)
 3. Confirm that filters from multiple categories can be applied and removed, both on the sidebar
 and the panel beneath the search form.
 
-If the `flipflop` gem has been updated, check that the `:gdt` feature is working by ensuring that
+Check that the `:geodata` feature is working by ensuring that
 UI elements specific to GDT (e.g., geospatial search fields or the 'Ask GIS' link) appear with the
 feature flag enabled, and do not when it is disabled.
 
@@ -91,8 +91,11 @@ See `Optional Environment Variables` for more information.
 `basic_search#index`.
 - `ACTIVE_FILTERS`: If populated, this list of strings defines which filters are shown to the user, and the order in which they appear. Values are case sensitive, and must match the corresponding aggregations used in the TIMDEX GraphQL query. Extraneous values will be ignored. If not populated, all filters will be shown.
 - `BOOLEAN_OPTIONS`: comma separated list of values to present to testers on instances where `BOOLEAN_PICKER` feature is enabled.
-- `BOOLEAN_PICKER`: feature to allow users to select their preferred boolean type. If set, feature is enabled. This feature is only intended for internal team
+- `FEATURE_BOOLEAN_PICKER`: feature to allow users to select their preferred boolean type. If set to `true`, feature is enabled. This feature is only intended for internal team
   testing and should never be enabled in production (mostly because the UI is a mess more than it would cause harm).
+- `FEATURE_GDT`: Enables features related to geospatial data discovery. Setting this variable to `true` will trigger geodata
+mode. Note that this is currently intended _only_ for the geodata app and
+may have unexpected consequences if applied to other TIMDEX UI apps.
 - `FILTER_ACCESS_TO_FILES`: The name to use instead of "Access to files" for that filter / aggregation.
 - `FILTER_CONTENT_TYPE`: The name to use instead of "Content type" for that filter / aggregation.
 - `FILTER_CONTRIBUTOR`: The name to use instead of "Contributor" for that filter / aggregation.
@@ -102,9 +105,6 @@ See `Optional Environment Variables` for more information.
 - `FILTER_PLACE`: The name to use instead of "Place" for that filter / aggregation.
 - `FILTER_SOURCE`: The name to use instead of "Source" for that filter / aggregation.
 - `FILTER_SUBJECT`: The name to use instead of "Subject" for that filter / aggregation.
-- `GDT`: Enables features related to geospatial data discovery. Setting this variable with any value will trigger GDT
-mode (e.g., `GDT=false` will still enable GDT features). Note that this is currently intended _only_ for the GDT app and
-may have unexpected consequences if applied to other TIMDEX UI apps.
 - `GLOBAL_ALERT`: The main functionality for this comes from our theme gem, but when set the value will be rendered as
   safe html above the main header of the site.
 - `ORIGINS`: sets origins for CORS (currently used only for TACOS API calls).

config/application.rb

@@ -11,14 +11,6 @@
 
 module TimdexUi
   class Application < Rails::Application
-    # Before filter for Flipflop dashboard. Replace with a lambda or method name
-    # defined in ApplicationController to implement access control.
-    config.flipflop.dashboard_access_filter = -> { head :forbidden }
-
-    # By default, when set to `nil`, strategy loading errors are suppressed in test
-    # mode. Set to `true` to always raise errors, or `false` to always warn.
-    config.flipflop.raise_strategy_errors = nil
-
     # Initialize configuration defaults for originally generated Rails version.
     config.load_defaults 7.2
 

config/environments/development.rb

@@ -1,14 +1,6 @@
 require "active_support/core_ext/integer/time"
 
 Rails.application.configure do
-  # Before filter for Flipflop dashboard. Replace with a lambda or method name
-  # defined in ApplicationController to implement access control.
-  config.flipflop.dashboard_access_filter = nil
-
-  # By default, when set to `nil`, strategy loading errors are suppressed in test
-  # mode. Set to `true` to always raise errors, or `false` to always warn.
-  config.flipflop.raise_strategy_errors = nil
-
   # Settings specified here will take precedence over those in config/application.rb.
 
   # In the development environment your application's code is reloaded any time

config/environments/test.rb

@@ -6,14 +6,6 @@
 # and recreated between test runs. Don't rely on the data there!
 
 Rails.application.configure do
-  # Before filter for Flipflop dashboard. Replace with a lambda or method name
-  # defined in ApplicationController to implement access control.
-  config.flipflop.dashboard_access_filter = nil
-
-  # By default, when set to `nil`, strategy loading errors are suppressed in test
-  # mode. Set to `true` to always raise errors, or `false` to always warn.
-  config.flipflop.raise_strategy_errors = nil
-
   # Settings specified here will take precedence over those in config/application.rb.
 
   # While tests run files are not watched, reloading is not necessary.

config/features.rb

@@ -1,5 +1,4 @@
 Rails.application.routes.draw do
-  mount Flipflop::Engine => "/flipflop"
   root "basic_search#index"
 
   get 'analyze', to: 'tacos#analyze'

config/routes.rb

@@ -0,0 +1,10 @@
+class DropFlipflipFeatures < ActiveRecord::Migration[7.2]
+  def change
+    drop_table :flipflop_features do |t|
+      t.string :key, null: false
+      t.boolean :enabled, null: false, default: false
+
+      t.timestamps null: false
+    end
+  end
+end