activity_notification 1.5.1 → 1.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6396ab94a172005c0bd940742471a388e746419e
-  data.tar.gz: 66f4b6b5162212ef397ed3fb26dfadbbe9709d97
+  metadata.gz: 24f6a30a0e5feec1ffcb217bf78fa3283f4e369b
+  data.tar.gz: efc4c998546752ccc9bca15a2d6d35e93d9e13eb
 SHA512:
-  metadata.gz: b3c26fad67e4e3647ac81f32e46181cfe1f420d8dd820b21f2f8f2a3039f87f200b5781a7eb203ec2a6f41d9d7a62c73aea249ee760a2de94341c987c3e16d92
-  data.tar.gz: f7d936e5c59608b736d068ee230c7919c6369ffd98c5c543343029e7fbfed58c611c2d17910ec74be2b398729a596217a02fda5c930a27b87843c1f55f81a892
+  metadata.gz: c30d75ab0300d5895d341fe7ca2f89b039a0cc5db82c9708b335409557aa49da457e81a74a29ad4c01b0d28f93a4abf5d7652effbaad4e0e58a337187361d743
+  data.tar.gz: 81698f2ec06b1cc880555a1103e24348f00ab2209621e47c3b2336dc0fb747ee96f765decca07b5e6ac230455927616d393c748efa90f31f294fa3dc8d0f618a

data/CHANGELOG.md

@@ -1,3 +1,14 @@
+## 1.6.0 / 2018-11-11
+[Full Changelog](http://github.com/simukappu/activity_notification/compare/v1.5.1...v1.6.0)
+
+Enhancements:
+
+* Add simple default routes with devise integration - [#64](https://github.com/simukappu/activity_notification/issues/64)
+
+Bug Fixes:
+
+* Update *Subscription.optional_targets* into HashWithIndifferentAccess to fix subscriptions with mongoid
+
 ## 1.5.1 / 2018-08-26
 [Full Changelog](http://github.com/simukappu/activity_notification/compare/v1.5.0...v1.5.1)
 

data/README.md

@@ -57,6 +57,7 @@
       - [Advanced notifiable path](#advanced-notifiable-path)
   - [Configuring views](#configuring-views)
   - [Configuring routes](#configuring-routes)
+    - [Routes with scope](#routes-with-scope)
   - [Creating notifications](#creating-notifications)
     - [Notification API](#notification-api)
     - [Automatic tracked notifications](#automatic-tracked-notifications)
@@ -83,6 +84,9 @@
     - [Managing subscriptions](#managing-subscriptions)
     - [Customizing subscriptions](#customizing-subscriptions)
   - [Integration with Devise](#integration-with-devise)
+    - [Configuring integration with Devise](#configuring-integration-with-devise)
+    - [Using different model as target](#using-different-model-as-target)
+    - [Configuring simple default routes](#configuring-simple-default-routes)
   - [Optional notification targets](#optional-notification-targets)
     - [Configuring optional targets](#configuring-optional-targets)
     - [Customizing message format](#customizing-message-format)
@@ -318,7 +322,22 @@ Rails.application.routes.draw do
 end
 ```
 
-Then, you can access several pages like *users/1/notifications* and manage open/unopen of notifications using **notifications_controller**.
+Then, you can access several pages like */users/1/notifications* and manage open/unopen of notifications using **notifications_controller**.
+If you use Devise integration and you want to configure simple default routes for authenticated users, see [Configuring simple default routes](#configuring-simple-default-routes).
+
+#### Routes with scope
+
+You can also configure *activity_notification* routes with scope like this:
+
+```ruby
+Rails.application.routes.draw do
+  scope :myscope, as: :myscope do
+    notify_to :users, routing_scope: :myscope
+  end
+end
+```
+
+Then, pages are shown as */myscope/users/1/notifications*.
 
 ### Creating notifications
 
@@ -914,7 +933,9 @@ If you would like to customize subscription controllers or views, you can use ge
 
 *activity_notification* supports to integrate with devise authentication.
 
-First, add **:with_devise** option in notification routing to *config/routes.rb* for the target:
+#### Configuring integration with Devise
+
+Add **:with_devise** option in notification routing to *config/routes.rb* for the target:
 
 ```ruby
 Rails.application.routes.draw do
@@ -928,6 +949,8 @@ Then *activity_notification* will use **notifications_with_devise_controller** a
 
 *Hint*: HTTP 403 Forbidden will be returned for unauthorized notifications.
 
+#### Using different model as target
+
 You can also use different model from Devise resource as a target. When you will add this to *config/routes.rb*:
 
 ```ruby
@@ -948,7 +971,32 @@ end
 ```
 
 *activity_notification* will authenticate *:admins* notifications with devise authentication for *:users*.
-In this example *activity_notification* will confirm the *user* who *admin* belongs to with authenticated user by devise.
+In this example, *activity_notification* will confirm *admin* belonging to authenticated *user* by Devise.
+
+#### Configuring simple default routes
+
+You can configure simple default routes for authenticated users, like */notifications* instead of */users/1/notifications*. Use *:devise_default_routes* option like this:
+
+```ruby
+Rails.application.routes.draw do
+  devise_for :users
+  notify_to :users, with_devise: :users, devise_default_routes: true
+end
+```
+
+If you use multiple notification targets with Devise, you can also use this option with scope like this:
+
+```ruby
+Rails.application.routes.draw do
+  devise_for :users
+  # Integrated with devise for different model, and use with scope
+  scope :admins, as: :admins do
+    notify_to :admins, with_devise: :users, devise_default_routes: true, routing_scope: :admins
+  end
+end
+```
+
+Then, you can access */admins/notifications* instead of */admins/1/notifications*.
 
 
 ### Optional notification targets

data/app/controllers/activity_notification/notifications_controller.rb

@@ -135,7 +135,7 @@ module ActivityNotification
                                params[:reverse].to_s.to_boolean(false) : nil
         with_group_members = params[:with_group_members].present? || params[:without_grouping].present? ?
                                params[:with_group_members].to_s.to_boolean(false) || params[:without_grouping].to_s.to_boolean(false) : nil
-        @index_options     = params.permit(:filter, :filtered_by_type, :filtered_by_group_type, :filtered_by_group_id, :filtered_by_key)
+        @index_options     = params.permit(:filter, :filtered_by_type, :filtered_by_group_type, :filtered_by_group_id, :filtered_by_key, :routing_scope, :devise_default_routes)
                                    .to_h.symbolize_keys
                                    .merge(limit: limit, reverse: reverse, with_group_members: with_group_members)
       end

data/app/controllers/activity_notification/subscriptions_controller.rb

@@ -52,6 +52,7 @@ module ActivityNotification
     #   @param [Hash] params Request parameters
     #   @return [Responce] HTML view as default
     def show
+      set_index_options
     end
   
     # Deletes a subscription.
@@ -190,7 +191,7 @@ module ActivityNotification
         limit          = params[:limit].to_i > 0 ? params[:limit].to_i : nil
         reverse        = params[:reverse].present? ?
                            params[:reverse].to_s.to_boolean(false) : nil
-        @index_options = params.permit(:filter, :filtered_by_key)
+        @index_options = params.permit(:filter, :filtered_by_key, :routing_scope, :devise_default_routes)
                                .to_h.symbolize_keys.merge(limit: limit, reverse: reverse)
       end
 

data/app/views/activity_notification/notifications/default/_default.html.erb

@@ -34,31 +34,31 @@
 
 <div class='<%= "notification_#{notification.id}" %>'>
   <% if notification.unopened? %>
-    <%= link_to open_notification_path_for(notification, reload: false), method: :post, remote: true, class: "unopened_wrapper" do %>
+    <%= link_to open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(reload: false)), method: :post, remote: true, class: "unopened_wrapper" do %>
       <div class="unopned_circle"></div>
       <div class="unopned_description_wrapper">
         <p class="unopned_description">Open</p>
       </div>
     <% end %>
-    <%= link_to open_notification_path_for(notification, move: true), method: :post do %>
+    <%= link_to open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(move: true)), method: :post do %>
       <%= yield :notification_content %>
     <% end %>
     <div class="unopened_wrapper"></div>
   <% else %>
-    <%= link_to move_notification_path_for(notification) do %>
+    <%= link_to move_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes)) do %>
       <%= yield :notification_content %>
     <% end %>
   <% end %>
 
-  <%#= link_to "Move", move_notification_path_for(notification) %>
+  <%#= link_to "Move", move_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes)) %>
   <%# if notification.unopened? %>
-    <%#= link_to "Open and move (GET)", move_notification_path_for(notification, open: true) %>
-    <%#= link_to "Open and move (POST)", open_notification_path_for(notification, move: true), method: :post %>
-    <%#= link_to "Open", open_notification_path_for(notification, index_options: @index_options), method: :post %>
-    <%#= link_to "Open (Ajax)", open_notification_path_for(notification, reload: false), method: :post, remote: true %>
+    <%#= link_to "Open and move (GET)", move_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(open: true)) %>
+    <%#= link_to "Open and move (POST)", open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(move: true)), method: :post %>
+    <%#= link_to "Open", open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(index_options: @index_options))), method: :post %>
+    <%#= link_to "Open (Ajax)", open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(reload: false)), method: :post, remote: true %>
   <%# end %>
-  <%#= link_to "Destroy", notification_path_for(notification, index_options: @index_options), method: :delete %>
-  <%#= link_to "Destroy (Ajax)", notification_path_for(notification, reload: false), method: :delete, remote: true %>
+  <%#= link_to "Destroy", notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(index_options: @index_options)), method: :delete %>
+  <%#= link_to "Destroy (Ajax)", notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(reload: false)), method: :delete, remote: true %>
 
 </div>
 

data/app/views/activity_notification/notifications/default/_default_without_grouping.html.erb

@@ -23,31 +23,31 @@
 
 <div class='<%= "notification_#{notification.id}" %>'>
   <% if notification.unopened? %>
-    <%= link_to open_notification_path_for(notification, reload: false, without_grouping: parameters[:with_group_members]), method: :post, remote: true, class: "unopened_wrapper" do %>
+    <%= link_to open_notification_path_for(notification, parameters.slice(:with_group_members, :routing_scope, :devise_default_routes).merge(reload: false), method: :post, remote: true, class: "unopened_wrapper" do %>
       <div class="unopned_circle"></div>
       <div class="unopned_description_wrapper">
         <p class="unopned_description">Open</p>
       </div>
     <% end %>
-    <%= link_to open_notification_path_for(notification, move: true), method: :post do %>
+    <%= link_to open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(move: true)), method: :post do %>
       <%= yield :notification_content %>
     <% end %>
     <div class="unopened_wrapper"></div>
   <% else %>
-    <%= link_to move_notification_path_for(notification) do %>
+    <%= link_to move_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes)) do %>
       <%= yield :notification_content %>
     <% end %>
   <% end %>
 
-  <%#= link_to "Move", move_notification_path_for(notification) %>
+  <%#= link_to "Move", move_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes)) %>
   <%# if notification.unopened? %>
-    <%#= link_to "Open and move (GET)", move_notification_path_for(notification, open: true) %>
-    <%#= link_to "Open and move (POST)", open_notification_path_for(notification, move: true), method: :post %>
-    <%#= link_to "Open", open_notification_path_for(notification, index_options: @index_options), method: :post %>
-    <%#= link_to "Open (Ajax)", open_notification_path_for(notification, reload: false, with_group_members: parameters[:with_group_members]), method: :post, remote: true %>
+    <%#= link_to "Open and move (GET)", move_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(open: true)) %>
+    <%#= link_to "Open and move (POST)", open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(move: true)), method: :post %>
+    <%#= link_to "Open", open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(index_options: @index_options)), method: :post %>
+    <%#= link_to "Open (Ajax)", open_notification_path_for(notification, parameters.slice(with_group_members:, :routing_scope, :devise_default_routes).merge(reload: false)), method: :post, remote: true %>
   <%# end %>
-  <%#= link_to "Destroy", notification_path_for(notification, index_options: @index_options), method: :delete %>
-  <%#= link_to "Destroy (Ajax)", notification_path_for(notification, reload: false, with_group_members: parameters[:with_group_members]), method: :delete, remote: true %>
+  <%#= link_to "Destroy", notification_path_for(notification, index_options: parameters.slice(:routing_scope, :devise_default_routes).merge(index_options: @index_options)), method: :delete %>
+  <%#= link_to "Destroy (Ajax)", notification_path_for(notification, parameters.slice(with_group_members:, :routing_scope, :devise_default_routes).merge(reload: false)), method: :delete, remote: true %>
 
 </div>
 

data/app/views/activity_notification/notifications/default/_index.html.erb

@@ -12,16 +12,16 @@
         Notifications
       </p>
       <p class="notification_header_menu">
-        <%= link_to "Open all", open_all_notifications_path_for(@target, parameters), method: :post, remote: true %>
+        <%= link_to "Open all", open_all_notifications_path_for(@target, parameters.slice(:routing_scope, :devise_default_routes)), method: :post, remote: true %>
         <% if @target.class.subscription_enabled? %>
-          <%= link_to "Subscriptions", subscriptions_path_for(@target) %>
+          <%= link_to "Subscriptions", subscriptions_path_for(@target, parameters.slice(:routing_scope, :devise_default_routes)) %>
         <% end %>
       </p>
     </div>
     <div class="notifications">
       <%= yield :notification_index %>
     </div>
-    <%= link_to notifications_path_for(@target) do %>
+    <%= link_to notifications_path_for(@target, parameters.slice(:routing_scope, :devise_default_routes)) do %>
       <div class="notification_link_wrapper">
         <p class="notification_link">
           See notifications

data/app/views/activity_notification/notifications/default/index.html.erb

@@ -1,20 +1,20 @@
 <div class="notification_wrapper">
   <div class="notification_header">
-    <h1>Notifications to <%= @target.printable_target_name %> <%= link_to open_all_notifications_path_for(@target, @index_options), method: :post, remote: true do %><span class="notification_count"><span class="<%= 'unopened' if @target.has_unopened_notifications?(@index_options) %>"><%= @target.unopened_notification_count(@index_options) %></span></span><% end %></h1>
+    <h1>Notifications to <%= @target.printable_target_name %> <%= link_to open_all_notifications_path_for(@target, @index_options.slice(:routing_scope, :devise_default_routes)), method: :post, remote: true do %><span class="notification_count"><span class="<%= 'unopened' if @target.has_unopened_notifications?(@index_options) %>"><%= @target.unopened_notification_count(@index_options) %></span></span><% end %></h1>
   </div>
   <div class="notifications">
     <% if @index_options[:with_group_members] %>
-      <%= render_notification @notifications, fallback: :default_without_grouping, with_group_members: true %>
+      <%= render_notification @notifications, @index_options.slice(:routing_scope, :devise_default_routes).merge(fallback: :default_without_grouping, with_group_members: true) %>
     <% else %>
-      <%= render_notification @notifications, fallback: :default %>
-      <%#= render_notification @notifications, fallback: :text %>
+      <%= render_notification @notifications, @index_options.slice(:routing_scope, :devise_default_routes).merge(fallback: :default) %>
+      <%#= render_notification @notifications, @index_options.merge(fallback: :text) %>
     <% end %>
   </div>
 </div>
 
-<%#= render_notifications_of @target, fallback: :default, index_content: :with_attributes %>
-<%#= render_notifications_of @target, fallback: :default, index_content: :unopened_with_attributes, reverse: true %>
-<%#= render_notifications_of @target, fallback: :default_without_grouping, index_content: :with_attributes, with_group_members: true %>
+<%#= render_notifications_of @target, @index_options.slice(:routing_scope, :devise_default_routes).merge(fallback: :default, index_content: :with_attributes) %>
+<%#= render_notifications_of @target, @index_options.slice(:routing_scope, :devise_default_routes).merge(fallback: :default, index_content: :unopened_with_attributes, reverse: true) %>
+<%#= render_notifications_of @target, @index_options.slice(:routing_scope, :devise_default_routes).merge(fallback: :default_without_grouping, index_content: :with_attributes, with_group_members: true) %>
 
 <style>
   .notification_wrapper .notification_header h1 span span{

data/app/views/activity_notification/notifications/default/open.js.erb

@@ -1,6 +1,6 @@
 $(".notification_count").html("<span class=\"<%= 'unopened' if @target.has_unopened_notifications?(@index_options) %>\"><%= @target.unopened_notification_count(@index_options) %></span>");
 <% if @index_options[:with_group_members] %>
-  $('<%= ".notification_#{@notification.id}" %>').html("<%= escape_javascript( render_notification(@notification, fallback: :default_without_grouping, with_group_members: true) ) %>");
+  $('<%= ".notification_#{@notification.id}" %>').html("<%= escape_javascript( render_notification(@notification, @index_options.slice(:routing_scope, :devise_default_routes).merge(fallback: :default_without_grouping, with_group_members: true)) ) %>");
 <% else %>
-  $('<%= ".notification_#{@notification.id}" %>').html("<%= escape_javascript( render_notification(@notification, fallback: :default) ) %>");
+  $('<%= ".notification_#{@notification.id}" %>').html("<%= escape_javascript( render_notification(@notification, @index_options.slice(:routing_scope, :devise_default_routes).merge(fallback: :default)) ) %>");
 <% end %>

data/app/views/activity_notification/notifications/default/open_all.js.erb

@@ -1,6 +1,6 @@
 $(".notification_count").html("<span class=\"<%= 'unopened' if @target.has_unopened_notifications?(@index_options) %>\"><%= @target.unopened_notification_count(@index_options) %></span>");
 <% if @index_options[:with_group_members] %>
-  $(".notifications").html("<%= escape_javascript( render_notification(@notifications, fallback: :default_without_grouping, with_group_members: true) ) %>");
+  $(".notifications").html("<%= escape_javascript( render_notification(@notifications, @index_options.slice(:routing_scope, :devise_default_routes).merge(fallback: :default_without_grouping, with_group_members: true)) ) %>");
 <% else %>
-  $(".notifications").html("<%= escape_javascript( render_notification(@notifications, fallback: :default) ) %>");
+  $(".notifications").html("<%= escape_javascript( render_notification(@notifications, @index_options.slice(:routing_scope, :devise_default_routes).merge(fallback: :default)) ) %>");
 <% end %>

data/app/views/activity_notification/notifications/default/show.html.erb

@@ -4,8 +4,8 @@
   </div>
   <ul>
     <div class="notifications">
-      <%= render_notification(@notification, fallback: :default) %>
-      <%#= render_notification @notification, fallback: :text %>
+      <%= render_notification @notification, @index_options.slice(:routing_scope, :devise_default_routes).merge(fallback: :default) %>
+      <%#= render_notification @notification, @index_options.slice(:routing_scope, :devise_default_routes).merge(fallback: :text) %>
     </div>
   </ul>
 </div>

data/app/views/activity_notification/optional_targets/default/base/_default.text.erb

@@ -3,7 +3,7 @@ Dear <%= @target.printable_target_name %>
 <%= @notification.notifier.present? ? @notification.notifier.printable_notifier_name : 'Someone' %> notified you of <%= @notification.notifiable.printable_notifiable_name(@notification.target) %><%= " in #{@notification.group.printable_group_name}" if @notification.group.present? %>.
 
 <%= "Move to notified #{@notification.notifiable.printable_type.downcase}:" %>
-  <%= move_notification_url_for(@notification, open: true) %>
+  <%= move_notification_url_for(@notification, parameters.slice(:routing_scope, :devise_default_routes).merge(open: true)) %>
 
 Thank you!
 

data/app/views/activity_notification/optional_targets/default/slack/_default.text.erb

@@ -3,4 +3,4 @@
 <%= @notification.notifier.present? ? @notification.notifier.printable_notifier_name : 'Someone' %> notified you of <%= @notification.notifiable.printable_notifiable_name(@notification.target) %><%= " in #{@notification.group.printable_group_name}" if @notification.group.present? %>.
 
 <%= "Move to notified #{@notification.notifiable.printable_type.downcase}:" %>
-  <%= move_notification_url_for(@notification, open: true) %>
+  <%= move_notification_url_for(@notification, parameters.slice(:routing_scope, :devise_default_routes).merge(open: true)) %>

data/app/views/activity_notification/subscriptions/default/_notification_keys.html.erb

@@ -9,7 +9,7 @@
               <%= key %>
             </h3>
             <p>
-              <%= link_to "Notifications", notifications_path_for(target, filtered_by_key: key) %>
+              <%= link_to "Notifications", notifications_path_for(target, option_params.merge(filtered_by_key: key)) %>
             </p>
           </div>
           <div class="field_wrapper subscribing">
@@ -97,7 +97,7 @@
         <%#= f.submit "Save" %>
       </div>
     <%# end %>
-<%#= link_to "See notifications", notifications_path_for(target, filtered_by_key: key) %>
+<%#= link_to "See notifications", notifications_path_for(target, option_params.merge(filtered_by_key: key)) %>
     <br/><br/>
   <%# end %>
 <%# else %>

data/app/views/activity_notification/subscriptions/default/_subscription.html.erb

@@ -5,7 +5,7 @@
     </h3>
 
     <p>
-      <%= link_to "Notifications", notifications_path_for(subscription.target, filtered_by_key: subscription.key) %>
+      <%= link_to "Notifications", notifications_path_for(subscription.target, option_params.merge(filtered_by_key: subscription.key)) %>
     </p>
   </div>
 
@@ -48,14 +48,14 @@
     <div class="field">
       <div class="ui checkbox">
         <% if subscription.subscribing_to_email? %>
-          <%= link_to unsubscribe_to_email_url_for(subscription, option_params), onclick: '$(this).find("input").prop("checked", false)', method: :post, remote: true do %>
+          <%= link_to unsubscribe_to_email_path_for(subscription, option_params), onclick: '$(this).find("input").prop("checked", false)', method: :post, remote: true do %>
             <label>
               <%= check_box :subscribing_to_email, "", { checked: true }, 'true', 'false' %>
               <div class="slider"></div>
             </label>
           <% end %>
         <% else %>
-          <%= link_to subscribe_to_email_url_for(subscription, option_params), onclick: '$(this).find("input").prop("checked", true)', method: :post, remote: true do %>
+          <%= link_to subscribe_to_email_path_for(subscription, option_params), onclick: '$(this).find("input").prop("checked", true)', method: :post, remote: true do %>
             <label>
               <%= check_box :subscribing_to_email, "", { checked: false }, 'true', 'false' %>
               <div class="slider"></div>
@@ -76,14 +76,14 @@
       <div class="field">
         <div class="ui checkbox">
           <% if subscription.subscribing_to_optional_target?(optional_target_name) %>
-            <%= link_to unsubscribe_to_optional_target_url_for(subscription, option_params.merge(optional_target_name: optional_target_name)), onclick: '$(this).find("input").prop("checked", false)', method: :post, remote: true do %>
+            <%= link_to unsubscribe_to_optional_target_path_for(subscription, option_params.merge(optional_target_name: optional_target_name)), onclick: '$(this).find("input").prop("checked", false)', method: :post, remote: true do %>
               <label>
                 <%= check_box optional_target_name, "", { checked: true }, 'true', 'false' %>
                 <div class="slider"></div>
               </label>
             <% end %>
           <% else %>
-            <%= link_to subscribe_to_optional_target_url_for(subscription, option_params.merge(optional_target_name: optional_target_name)), onclick: '$(this).find("input").prop("checked", true)', method: :post, remote: true do %>
+            <%= link_to subscribe_to_optional_target_path_for(subscription, option_params.merge(optional_target_name: optional_target_name)), onclick: '$(this).find("input").prop("checked", true)', method: :post, remote: true do %>
               <label>
                 <%= check_box optional_target_name, "", { checked: false }, 'true', 'false' %>
                 <div class="slider"></div>

data/app/views/activity_notification/subscriptions/default/show.html.erb

@@ -4,7 +4,7 @@
   </div>
   <div class="fields_area">
     <div class="subscription" id="subscription">
-      <%= render 'subscription', subscription: @subscription, option_params: {} %>
+      <%= render 'subscription', subscription: @subscription, option_params: @index_options %>
     </div>
   </div>
 </div>

data/app/views/activity_notification/subscriptions/default/subscribe.js.erb

@@ -1,6 +1,6 @@
 setTimeout(function () {
   $("#subscriptions").html("<%= escape_javascript( render 'subscriptions', subscriptions: @subscriptions, option_params: @index_options ) %>");
-  $("#subscription").html("<%= escape_javascript( render 'subscription', subscription: @subscription, option_params: {} ) %>");
+  $("#subscription").html("<%= escape_javascript( render 'subscription', subscription: @subscription, option_params: @index_options ) %>");
 }, 400);
 $("#notification_keys").html("<%= escape_javascript( render 'notification_keys', target: @target, notification_keys: @notification_keys, option_params: @index_options ) %>");
 $("#subscription_form").html("<%= escape_javascript( render 'form', target: @target, option_params: @index_options ) %>");

data/app/views/activity_notification/subscriptions/default/subscribe_to_email.js.erb

@@ -1,5 +1,5 @@
 $("#subscriptions").html("<%= escape_javascript( render 'subscriptions', subscriptions: @subscriptions, option_params: @index_options ) %>");
-$("#subscription").html("<%= escape_javascript( render 'subscription', subscription: @subscription, option_params: {} ) %>");
+$("#subscription").html("<%= escape_javascript( render 'subscription', subscription: @subscription, option_params: @index_options ) %>");
 $("#notification_keys").html("<%= escape_javascript( render 'notification_keys', target: @target, notification_keys: @notification_keys, option_params: @index_options ) %>");
 $("#subscription_form").html("<%= escape_javascript( render 'form', target: @target, option_params: @index_options ) %>");
 

data/app/views/activity_notification/subscriptions/default/subscribe_to_optional_target.js.erb

@@ -1,5 +1,5 @@
 $("#subscriptions").html("<%= escape_javascript( render 'subscriptions', subscriptions: @subscriptions, option_params: @index_options ) %>");
-$("#subscription").html("<%= escape_javascript( render 'subscription', subscription: @subscription, option_params: {} ) %>");
+$("#subscription").html("<%= escape_javascript( render 'subscription', subscription: @subscription, option_params: @index_options ) %>");
 $("#notification_keys").html("<%= escape_javascript( render 'notification_keys', target: @target, notification_keys: @notification_keys, option_params: @index_options ) %>");
 $("#subscription_form").html("<%= escape_javascript( render 'form', target: @target, option_params: @index_options ) %>");
 

data/app/views/activity_notification/subscriptions/default/unsubscribe.js.erb

@@ -1,6 +1,6 @@
 setTimeout(function () {
   $("#subscriptions").html("<%= escape_javascript( render 'subscriptions', subscriptions: @subscriptions, option_params: @index_options ) %>");
-  $("#subscription").html("<%= escape_javascript( render 'subscription', subscription: @subscription, option_params: {} ) %>");
+  $("#subscription").html("<%= escape_javascript( render 'subscription', subscription: @subscription, option_params: @index_options ) %>");
 }, 400);
 $("#notification_keys").html("<%= escape_javascript( render 'notification_keys', target: @target, notification_keys: @notification_keys, option_params: @index_options ) %>");
 $("#subscription_form").html("<%= escape_javascript( render 'form', target: @target, option_params: @index_options ) %>");

data/app/views/activity_notification/subscriptions/default/unsubscribe_to_email.js.erb

@@ -1,5 +1,5 @@
 $("#subscriptions").html("<%= escape_javascript( render 'subscriptions', subscriptions: @subscriptions, option_params: @index_options ) %>");
-$("#subscription").html("<%= escape_javascript( render 'subscription', subscription: @subscription, option_params: {} ) %>");
+$("#subscription").html("<%= escape_javascript( render 'subscription', subscription: @subscription, option_params: @index_options ) %>");
 $("#notification_keys").html("<%= escape_javascript( render 'notification_keys', target: @target, notification_keys: @notification_keys, option_params: @index_options ) %>");
 $("#subscription_form").html("<%= escape_javascript( render 'form', target: @target, option_params: @index_options ) %>");
 

data/app/views/activity_notification/subscriptions/default/unsubscribe_to_optional_target.js.erb

@@ -1,5 +1,5 @@
 $("#subscriptions").html("<%= escape_javascript( render 'subscriptions', subscriptions: @subscriptions, option_params: @index_options ) %>");
-$("#subscription").html("<%= escape_javascript( render 'subscription', subscription: @subscription, option_params: {} ) %>");
+$("#subscription").html("<%= escape_javascript( render 'subscription', subscription: @subscription, option_params: @index_options ) %>");
 $("#notification_keys").html("<%= escape_javascript( render 'notification_keys', target: @target, notification_keys: @notification_keys, option_params: @index_options ) %>");
 $("#subscription_form").html("<%= escape_javascript( render 'form', target: @target, option_params: @index_options ) %>");