Extract the notification poll interval to an engine configuration set…

…ting
samvera · May 12, 2017 · 4c60143 · 4c60143
1 parent a8ee769
commit 4c60143
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/app/views/hyrax/users/_notify_number.html.erb b/app/views/hyrax/users/_notify_number.html.erb
@@ -2,7 +2,7 @@
             'aria-label' => t("hyrax.toolbar.notifications"),
             class: "notify-number",
             data: { 'update-poll-url' => hyrax.user_notify_path,
-                    'update-poll-interval' => 30 } do %>
+                    'update-poll-interval' => Hyrax.config.notifications_update_poll_interval } do %>
     <span class="fa fa-bell"></span>
     <span class="count invisible label label-danger">0</span>
 <% end %>
diff --git a/lib/generators/hyrax/templates/config/hyrax.rb b/lib/generators/hyrax/templates/config/hyrax.rb
@@ -20,6 +20,9 @@
   # How many notifications should be displayed on the dashboard
   # config.max_notifications_for_dashboard = 5
 
+  # How often clients should poll for notifications
+  # config.notifications_update_poll_interval = 30.seconds
+
   # How frequently should a file be fixity checked
   # config.max_days_between_fixity_checks = 7
 

diff --git a/lib/hyrax/configuration.rb b/lib/hyrax/configuration.rb
@@ -304,6 +304,11 @@ def max_notifications_for_dashboard
       @max_notifications_for_dashboard ||= 5
     end
 
+    attr_writer :notifications_update_poll_interval
+    def notifications_update_poll_interval
+      @notifications_update_poll_interval ||= 30.seconds
+    end
+
     attr_writer :activity_to_show_default_seconds_since_now
     def activity_to_show_default_seconds_since_now
       @activity_to_show_default_seconds_since_now ||= 24 * 60 * 60