Merge pull request #28 from seantaylor/fix/better-cancel-in-admin

Small UI change to canceling in the admin
solidusio · Aug 11, 2017 · 8d81b64 · 8d81b64
2 parents 8e92e86 + d5e486a
commit 8d81b64
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/app/views/spree/admin/solidus_subscriptions/subscriptions/_subscription.html.erb b/app/views/spree/admin/solidus_subscriptions/subscriptions/_subscription.html.erb
@@ -27,11 +27,12 @@
     <% if subscription.state_events.include?(:cancel) %>
       <%=
         link_to_with_icon(
-          :pause,
+          :stop,
           t('.cancel'),
           spree.cancel_admin_subscription_path(subscription),
           no_text: true,
-          method: :delete
+          method: :delete,
+          data: { confirm: t('.cancel_alert') }
         )
       %>
     <% end %>

diff --git a/config/locales/en.yml b/config/locales/en.yml
@@ -23,6 +23,7 @@ en:
         subscriptions:
           subscription:
             cancel: Cancel
+            cancel_alert: "Are you sure you want to cancel this subscription?"
             activate: Activate
             skip: Skip One