diff options
| author | jessib <jessib@riseup.net> | 2013-10-10 11:35:26 -0700 |
|---|---|---|
| committer | jessib <jessib@riseup.net> | 2013-10-10 11:35:26 -0700 |
| commit | 51f93fc87c9cadbe52877ddc3e7c5fd07866b397 (patch) | |
| tree | 86098fd4f837151e8e292fbd39d92b1fc1f7819a /billing/app/views | |
| parent | 5627b01230cdfa918ad144b26183b4c0bf151360 (diff) | |
Admins can cancel pastdue subscriptions, but users cannot cancel their own pastdue subscription, as then admins won't be able to search for them.
Diffstat (limited to 'billing/app/views')
| -rw-r--r-- | billing/app/views/subscriptions/show.html.haml | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/billing/app/views/subscriptions/show.html.haml b/billing/app/views/subscriptions/show.html.haml index b258e47..f4d644a 100644 --- a/billing/app/views/subscriptions/show.html.haml +++ b/billing/app/views/subscriptions/show.html.haml @@ -3,4 +3,4 @@ Current Subscription = render :partial => "subscription_details", :locals => {:subscription => @subscription} -= link_to t(:cancel_subscription), user_subscription_path(@user, @subscription.id), :confirm => t(:are_you_sure), :method => :delete, :class => 'btn btn-danger' if ['Active', 'Pending'].include? @subscription.status # permission check or should that just be on show? # should you be able to cancel pending subscription? += link_to t(:cancel_subscription), user_subscription_path(@user, @subscription.id), :confirm => t(:are_you_sure), :method => :delete, :class => 'btn btn-danger' if ['Active', 'Pending'].include? @subscription.status or admin? # permission check or should that just be on show? # should you be able to cancel pending subscription? |