Merge pull request #320 from MITLibraries/use-253

Add feature flag for full record fulfillment links

Author: jazairi

README.md

@@ -97,6 +97,7 @@ See `Optional Environment Variables` for more information.
 - `FEATURE_GEODATA`: 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.
+- `FEATURE_RECORD_LINK`: Display the 'View full record' link below each record.
 - `FEATURE_SIMULATE_SEARCH_LATENCY`: DO NOT SET IN PRODUCTION. Set to ensure a minimum of a one second delay in returning search results. Useful to see spinners/loaders. Only introduces delay for results that take less than one second to complete.
 - `FEATURE_TAB_PRIMO_ALL`: Display a tab for displaying the combined Primo data (CDI + Alma)
 - `FEATURE_TAB_TIMDEX_ALL`: Display a tab for displaying the combined TIMDEX data. `TIMDEX_INDEX` affects which data appears in this tab.

app/models/feature.rb

@@ -34,7 +34,7 @@
 class Feature
   # List of all valid features in the application
   VALID_FEATURES = %i[geodata boolean_picker simulate_search_latency tab_primo_all tab_timdex_all
-                      tab_timdex_alma].freeze
+                      tab_timdex_alma record_link].freeze
 
   # Check if a feature is enabled by name
   #

app/views/search/_result.html.erb

@@ -56,8 +56,10 @@
       <%= render partial: 'search/highlights', locals: { result: result } %>
     </div>
 
-    <div class="result-get">
-      <%= view_record(result) %>
-    </div>
+    <% if Feature.enabled?(:record_link) %>
+      <div class="result-get">
+        <%= view_record(result) %>
+      </div>
+    <% end %>
   </div>
 </li>

app/views/search/_result_primo.html.erb

@@ -63,7 +63,9 @@
       <% if result[:links].present? %>
         <% result[:links].each do |link| %>
           <% if link['kind'].downcase == 'full record' %>
-            <%= view_record(link) %>
+            <% if Feature.enabled?(:record_link) %>
+              <%= link_to 'View full record', link['url'], class: 'button' %>
+            <% end %>
           <% else %>
             <%= link_to link['kind'].titleize, link['url'], class: 'button' %>
           <% end %>