activity_notification 2.0.0 → 2.1.0

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

@@ -7,20 +7,20 @@
         <strong><%= notification.notifier.present? ? notification.notifier.printable_notifier_name : 'Someone' %></strong>
         <% if notification.group_member_notifier_exists? %>
           <%= " and #{notification.group_member_notifier_count} other" %>
-          <%= notification.notifier.present? ? notification.notifier.printable_type.pluralize.downcase : 'people' %>
+          <%= notification.notifier.present? ? notification.notifier.printable_type.downcase.pluralize(notification.group_member_notifier_count) : 'people' %>
         <% end %>
         notified you of
         <% if notification.notifiable.present? %>
           <% if notification.group_member_exists? %>
-            <%= " #{notification.group_notification_count} #{notification.notifiable_type.humanize.pluralize.downcase} including" %>
+            <%= " #{notification.group_notification_count} #{notification.notifiable_type.humanize.downcase.pluralize(notification.group_notification_count)} including" %>
           <% end %>
           <%= notification.notifiable.printable_notifiable_name(notification.target) %>
           <%= "in #{notification.group.printable_group_name}" if notification.group.present? %>
         <% else %>
           <% if notification.group_member_exists? %>
-            <%= " #{notification.group_notification_count} #{notification.notifiable_type.humanize.pluralize.downcase}" %>
+            <%= " #{notification.group_notification_count} #{notification.notifiable_type.humanize.downcase.pluralize(notification.group_notification_count)}" %>
           <% else %>
-            <%= " a #{notification.notifiable_type.humanize.singularize.downcase}" %>
+            <%= " a #{notification.notifiable_type.humanize.downcase.singularize}" %>
           <% end %>
           <%= "in #{notification.group.printable_group_name}" if notification.group.present? %>
           but the notifiable is not found. It may have been deleted.
@@ -34,13 +34,13 @@
 
 <div class='<%= "notification_#{notification.id}" %>'>
   <% if notification.unopened? %>
-    <%= 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>
+    <%= link_to open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(reload: false)), method: :put, remote: true, class: "unopened_wrapper" do %>
+      <div class="unopened_circle"></div>
+      <div class="unopened_description_wrapper">
+        <p class="unopened_description">Open</p>
       </div>
     <% end %>
-    <%= link_to open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(move: true)), method: :post do %>
+    <%= link_to open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(move: true)), method: :put do %>
       <%= yield :notification_content %>
     <% end %>
     <div class="unopened_wrapper"></div>
@@ -53,9 +53,9 @@
   <%#= 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, 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 %>
+    <%#= link_to "Open and move (PUT)", open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(move: true)), method: :put %>
+    <%#= link_to "Open", open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(index_options: @index_options))), method: :put %>
+    <%#= link_to "Open (Ajax)", open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(reload: false)), method: :put, remote: true %>
   <%# end %>
   <%#= 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 %>
@@ -63,13 +63,13 @@
 </div>
 
 <style>
-  /* unopned_circle */
+  /* unopened_circle */
   .unopened_wrapper{
     position: absolute;
     margin-top: 20px;
     margin-left: 56px;
   }
-  .unopened_wrapper .unopned_circle {
+  .unopened_wrapper .unopened_circle {
     display: block;
     width: 10px;
     height: 10px;
@@ -78,16 +78,16 @@
     background-color: #27a5eb;
     z-index: 2;
   }
-  .unopened_wrapper:hover > .unopned_description_wrapper{
+  .unopened_wrapper:hover > .unopened_description_wrapper{
     display: block;
   }
-  .unopened_wrapper .unopned_description_wrapper {
+  .unopened_wrapper .unopened_description_wrapper {
     display: none;
     position: absolute;
     margin-top: 26px;
     margin-left: -24px;
   }
-  .unopened_wrapper .unopned_description_wrapper .unopned_description {
+  .unopened_wrapper .unopened_description_wrapper .unopened_description {
     position: absolute;
     color: #fff;
     font-size: 12px;
@@ -98,7 +98,7 @@
     padding: 4px 12px;
     z-index: 999;
   }
-  .unopened_wrapper .unopned_description_wrapper .unopned_description:before {
+  .unopened_wrapper .unopened_description_wrapper .unopened_description:before {
      border: solid transparent;
      border-top-width: 0;
      content: "";

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

@@ -23,13 +23,13 @@
 
 <div class='<%= "notification_#{notification.id}" %>'>
   <% if notification.unopened? %>
-    <%= 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>
+    <%= link_to open_notification_path_for(notification, parameters.slice(:with_group_members, :routing_scope, :devise_default_routes).merge(reload: false)), method: :put, remote: true, class: "unopened_wrapper" do %>
+      <div class="unopened_circle"></div>
+      <div class="unopened_description_wrapper">
+        <p class="unopened_description">Open</p>
       </div>
     <% end %>
-    <%= link_to open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(move: true)), method: :post do %>
+    <%= link_to open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(move: true)), method: :put do %>
       <%= yield :notification_content %>
     <% end %>
     <div class="unopened_wrapper"></div>
@@ -42,9 +42,9 @@
   <%#= 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, 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 %>
+    <%#= link_to "Open and move (PUT)", open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(move: true)), method: :put %>
+    <%#= link_to "Open", open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(index_options: @index_options)), method: :put %>
+    <%#= link_to "Open (Ajax)", open_notification_path_for(notification, parameters.slice(with_group_members:, :routing_scope, :devise_default_routes).merge(reload: false)), method: :put, remote: true %>
   <%# end %>
   <%#= 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 %>
@@ -52,13 +52,13 @@
 </div>
 
 <style>
-  /* unopned_circle */
+  /* unopened_circle */
   .unopened_wrapper{
     position: absolute;
     margin-top: 20px;
     margin-left: 56px;
   }
-  .unopened_wrapper .unopned_circle {
+  .unopened_wrapper .unopened_circle {
     display: block;
     width: 10px;
     height: 10px;
@@ -67,16 +67,16 @@
     background-color: #27a5eb;
     z-index: 2;
   }
-  .unopened_wrapper:hover > .unopned_description_wrapper{
+  .unopened_wrapper:hover > .unopened_description_wrapper{
     display: block;
   }
-  .unopened_wrapper .unopned_description_wrapper {
+  .unopened_wrapper .unopened_description_wrapper {
     display: none;
     position: absolute;
     margin-top: 26px;
     margin-left: -24px;
   }
-  .unopened_wrapper .unopned_description_wrapper .unopned_description {
+  .unopened_wrapper .unopened_description_wrapper .unopened_description {
     position: absolute;
     color: #fff;
     font-size: 12px;
@@ -87,7 +87,7 @@
     padding: 4px 12px;
     z-index: 999;
   }
-  .unopened_wrapper .unopned_description_wrapper .unopned_description:before {
+  .unopened_wrapper .unopened_description_wrapper .unopened_description:before {
      border: solid transparent;
      border-top-width: 0;
      content: "";

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

@@ -69,12 +69,12 @@
           }
           // Push notificaion using Web Notification API by Push.js
           Push.create('ActivityNotification', {
-              body: notification.text,
-              timeout: 5000,
-              onClick: function () {
-                  location.href = notification.notifiable_path;
-                  this.close();
-              }
+            body: notification.text,
+            timeout: 5000,
+            onClick: function () {
+              location.href = notification.notifiable_path;
+              this.close();
+            }
           });
         }
       }

data/app/views/activity_notification/optional_targets/default/action_cable_channel/_default.html.erb

@@ -0,0 +1,176 @@
+<% content_for :notification_content, flush: true do %>
+  <div class='notification_list <%= notification.opened? ? "opened" : "unopened" %>'>
+    <div class="notification_list_cover"></div>
+    <div class="list_image"></div>
+    <div class="list_text_wrapper">
+      <p class="list_text">
+        <strong><%= notification.notifier.present? ? notification.notifier.printable_notifier_name : 'Someone' %></strong>
+        <% if notification.group_member_notifier_exists? %>
+          <%= " and #{notification.group_member_notifier_count} other" %>
+          <%= notification.notifier.present? ? notification.notifier.printable_type.pluralize.downcase : 'people' %>
+        <% end %>
+        notified you of
+        <% if notification.notifiable.present? %>
+          <% if notification.group_member_exists? %>
+            <%= " #{notification.group_notification_count} #{notification.notifiable_type.humanize.pluralize.downcase} including" %>
+          <% end %>
+          <%= notification.notifiable.printable_notifiable_name(notification.target) %>
+          <%= "in #{notification.group.printable_group_name}" if notification.group.present? %>
+        <% else %>
+          <% if notification.group_member_exists? %>
+            <%= " #{notification.group_notification_count} #{notification.notifiable_type.humanize.pluralize.downcase}" %>
+          <% else %>
+            <%= " a #{notification.notifiable_type.humanize.singularize.downcase}" %>
+          <% end %>
+          <%= "in #{notification.group.printable_group_name}" if notification.group.present? %>
+          but the notifiable is not found. It may have been deleted.
+        <% end %>
+        <br>
+        <span><%= notification.created_at.strftime("%b %d %H:%M") %></span>
+      </p>
+    </div>
+  </div>
+<% end %>
+
+<div class='<%= "notification_#{notification.id}" %>'>
+  <% if notification.unopened? %>
+    <%= link_to open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(reload: false)), method: :put, remote: true, class: "unopened_wrapper" do %>
+      <div class="unopened_circle"></div>
+      <div class="unopened_description_wrapper">
+        <p class="unopened_description">Open</p>
+      </div>
+    <% end %>
+    <%= link_to open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(move: true)), method: :put do %>
+      <%= yield :notification_content %>
+    <% end %>
+    <div class="unopened_wrapper"></div>
+  <% else %>
+    <%= 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, parameters.slice(:routing_scope, :devise_default_routes)) %>
+  <%# if notification.unopened? %>
+    <%#= 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 (PUT)", open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(move: true)), method: :put %>
+    <%#= link_to "Open", open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(index_options: @index_options))), method: :put %>
+    <%#= link_to "Open (Ajax)", open_notification_path_for(notification, parameters.slice(:routing_scope, :devise_default_routes).merge(reload: false)), method: :put, remote: true %>
+  <%# end %>
+  <%#= 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>
+
+<style>
+  /* unopened_circle */
+  .unopened_wrapper{
+    position: absolute;
+    margin-top: 20px;
+    margin-left: 56px;
+  }
+  .unopened_wrapper .unopened_circle {
+    display: block;
+    width: 10px;
+    height: 10px;
+    position: absolute;
+    border-radius: 50%;
+    background-color: #27a5eb;
+    z-index: 2;
+  }
+  .unopened_wrapper:hover > .unopened_description_wrapper{
+    display: block;
+  }
+  .unopened_wrapper .unopened_description_wrapper {
+    display: none;
+    position: absolute;
+    margin-top: 26px;
+    margin-left: -24px;
+  }
+  .unopened_wrapper .unopened_description_wrapper .unopened_description {
+    position: absolute;
+    color: #fff;
+    font-size: 12px;
+    text-align: center;
+
+    border-radius: 4px;
+    background: rgba(0, 0, 0, 0.8);
+    padding: 4px 12px;
+    z-index: 999;
+  }
+  .unopened_wrapper .unopened_description_wrapper .unopened_description:before {
+     border: solid transparent;
+     border-top-width: 0;
+     content: "";
+     display: block;
+     position: absolute;
+     width: 0;
+     left: 50%;
+     top: -5px;
+     margin-left: -5px;
+     height: 0;
+     border-width: 0 5px 5px 5px;
+     border-color: transparent transparent rgba(0, 0, 0, 0.8) transparent;
+     z-index: 0;
+  }
+
+  /* list */
+  .notification_list {
+    padding: 15px 10px;
+    position: relative;
+    border-bottom: 1px solid #e5e5e5;
+  }
+  .notification_list.unopened {
+    background-color: #eeeff4;
+  }
+  .notification_list:hover {
+    background-color: #f8f9fb;
+  }
+  .notification_list:last-child {
+    border-bottom: none;
+  }
+  .notification_list:after{
+    content: "";
+    clear: both;
+    display: block;
+  }
+  .notification_list .notification_list_cover{
+    position: absolute;
+    opacity: 0;
+    top: 0;
+    left: 0;
+    width: 100%;
+    height: 100%;
+    z-index: 1;
+
+  }
+  .notification_list .list_image {
+    float: left;
+    width: 40px;
+    height: 40px;
+    background-position: center;
+    background-repeat: no-repeat;
+    background-size: cover;
+    background-color: #979797;
+  }
+  .notification_list .list_text_wrapper {
+    float: left;
+    width: calc(100% - 60px);
+    margin-left: 20px;
+  }
+  .notification_list .list_text_wrapper .list_text {
+    color: #4f4f4f;
+    font-size: 14px;
+    line-height: 1.4;
+    margin-top: 0;
+    height: auto;
+    font-weight: normal;
+  }
+  .notification_list .list_text_wrapper .list_text strong{
+    font-weight: bold;
+  }
+  .notification_list .list_text_wrapper .list_text span {
+    color: #979797;
+    font-size: 13px;
+  }
+</style>

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

@@ -1,6 +1,6 @@
 <div class="fields_area">
   <div class="fields_wrapper">
-    <%= form_for(ActivityNotification::Subscription.new, as: :subscription, url: subscriptions_url_for(target, option_params), data: { remote: true }) do |f| %>
+    <%= form_for(ActivityNotification::Subscription.new, as: :subscription, url: subscriptions_url_for(target, option_params), data: { remote: true }, namespace: :new) do |f| %>
       <div class="field_wrapper">
         <div class="field_label">
           <%= f.label :key, "Notification key" %>

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

@@ -2,7 +2,7 @@
   <div class="fields_area">
     <% notification_keys.each do |key| %>
       <div class="fields_wrapper">
-        <%= form_for(ActivityNotification::Subscription.new, as: :subscription, url: subscriptions_url_for(target, option_params), data: { remote: true }) do |f| %>
+        <%= form_for(ActivityNotification::Subscription.new, as: :subscription, url: subscriptions_url_for(target, option_params), data: { remote: true }, namespace: key) do |f| %>
           <%= f.hidden_field :key, value: key %>
           <div class="fields_title_wrapper">
             <h3 class="fields_title">
@@ -55,8 +55,8 @@
               <div class="field">
                 <div class="ui checkbox">
                   <label>
-                    <%= hidden_field_tag "subscription[optional_targets][#{ActivityNotification::Subscription.to_optional_target_key(optional_target_name)}]", 'false' %>
-                    <%= check_box_tag "subscription[optional_targets][#{ActivityNotification::Subscription.to_optional_target_key(optional_target_name)}]", 'true', ActivityNotification.config.subscribe_as_default %>
+                    <%= hidden_field_tag "subscription[optional_targets][#{ActivityNotification::Subscription.to_optional_target_key(optional_target_name)}]", 'false', id: "#{key}_subscription_optional_targets_subscribing_to_#{ActivityNotification::Subscription.to_optional_target_key(optional_target_name)}_hidden" %>
+                    <%= check_box_tag "subscription[optional_targets][#{ActivityNotification::Subscription.to_optional_target_key(optional_target_name)}]", 'true', ActivityNotification.config.subscribe_as_default, id: "#{key}_subscription_optional_targets_subscribing_to_#{ActivityNotification::Subscription.to_optional_target_key(optional_target_name)}_check_box" %>
                     <div class="slider"></div>
                   </label>
                 </div>
@@ -78,31 +78,3 @@
     </div>
   </div>
 <% end %>
-
-<!--
-<%# if notification_keys.present? %>
-<%# notification_keys.each do |key| %>
-    Key: <%#= key %>
-<%#= form_for(ActivityNotification::Subscription.new, as: :subscription, url: subscriptions_url_for(target, option_params), data: {remote: true}) do |f| %>
-<%#= f.hidden_field :key, value: key %>
-      <div class="field">
-        <%#= f.label :subscribing, "Subscribe to this notification?" %>
-<%#= f.check_box :subscribing, {checked: true}, 'true', 'false' %>
-      </div>
-      <div class="field">
-        <%#= f.label :subscribing_to_email, "Subscribe to this notification email?" %>
-<%#= f.check_box :subscribing_to_email, {checked: true}, 'true', 'false' %>
-      </div>
-      <div class="actions">
-        <%#= f.submit "Save" %>
-      </div>
-    <%# end %>
-<%#= link_to "See notifications", notifications_path_for(target, option_params.merge(filtered_by_key: key)) %>
-    <br/><br/>
-  <%# end %>
-<%# else %>
-  No notification keys are available.
-<%# end %>
-<br/><br/>
--->
-

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

@@ -18,18 +18,18 @@
     <div class="field">
       <div class="ui checkbox">
         <% if subscription.subscribing? %>
-          <%= link_to unsubscribe_path_for(subscription, option_params), onclick: '$(this).find("input").prop("checked", false);$(this).parent().parent().parent().next().slideUp();;$(this).parent().parent().parent().next().next().slideUp();', method: :post, remote: true do %>
+          <%= link_to unsubscribe_path_for(subscription, option_params), onclick: '$(this).find("input").prop("checked", false);$(this).parent().parent().parent().next().slideUp();;$(this).parent().parent().parent().next().next().slideUp();', method: :put, remote: true do %>
             <%= check_box :subscribing, "", { checked: true }, 'true', 'false' %>
             <div class="slider"></div>
           <% end %>
         <% else %>
           <% if ActivityNotification.config.subscribe_as_default %>
-            <%= link_to subscribe_path_for(subscription, option_params), onclick: '$(this).find("input").prop("checked", true);$(this).parent().parent().parent().next().slideDown();$(this).parent().parent().parent().next().find("input").prop("checked", true);$(this).parent().parent().parent().next().next().slideDown();$(this).parent().parent().parent().next().next().find("input").prop("checked", true);', method: :post, remote: true do %>
+            <%= link_to subscribe_path_for(subscription, option_params), onclick: '$(this).find("input").prop("checked", true);$(this).parent().parent().parent().next().slideDown();$(this).parent().parent().parent().next().find("input").prop("checked", true);$(this).parent().parent().parent().next().next().slideDown();$(this).parent().parent().parent().next().next().find("input").prop("checked", true);', method: :put, remote: true do %>
               <%= check_box :subscribing, "", { checked: false }, 'true', 'false' %>
               <div class="slider"></div>
             <% end %>
           <% else %>
-            <%= link_to subscribe_path_for(subscription, option_params.merge(with_email_subscription: false)), onclick: '$(this).find("input").prop("checked", true);$(this).parent().parent().parent().next().slideDown();$(this).parent().parent().parent().next().next().slideDown();', method: :post, remote: true do %>
+            <%= link_to subscribe_path_for(subscription, option_params.merge(with_email_subscription: false)), onclick: '$(this).find("input").prop("checked", true);$(this).parent().parent().parent().next().slideDown();$(this).parent().parent().parent().next().next().slideDown();', method: :put, remote: true do %>
               <%= check_box :subscribing, "", { checked: false }, 'true', 'false' %>
               <div class="slider"></div>
             <% end %>
@@ -48,14 +48,14 @@
     <div class="field">
       <div class="ui checkbox">
         <% if subscription.subscribing_to_email? %>
-          <%= link_to unsubscribe_to_email_path_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: :put, 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_path_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: :put, 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_path_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: :put, 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_path_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: :put, remote: true do %>
               <label>
                 <%= check_box optional_target_name, "", { checked: false }, 'true', 'false' %>
                 <div class="slider"></div>