summaryrefslogtreecommitdiff
path: root/lib/solidus_subscriptions
diff options
context:
space:
mode:
authorSean <seand7565@gmail.com>2020-12-11 10:58:20 -0600
committerSean <seand7565@gmail.com>2020-12-14 09:26:57 -0600
commit595891ac6a7f6641b04f5cd4e1ecaa21c0ed7eb6 (patch)
treee737b463dfcba8f726e21b9c3e3465f6a2b4f96f /lib/solidus_subscriptions
parentd4743d79536be77bcf1fa0fb5ff055ba6145396a (diff)
Update cancel method on subscription to respect minimum date
Previously, the cancel method would check if the minimum cancellation date minus the current day is in the future before cancelling. However, that adds one day to whatever minimum cancellation date that you set - if you set one day, you would expect the user not to be able to cancel on the day that the subscription ships, but because the method checks if the date is in the future, the user can't cancel the day of, or the day before. This checks if the date is in the future OR if it's today, which removes the added day from the minimum_cancellation_notice. Also this sets the default to 0.days, as 1.day is pretty arbitrary Lastly, this locks canonical-rails down to 0.2.9, because Solidus still uses the `whitelisted_attributes` method, which was renamed in 0.2.10
Diffstat (limited to 'lib/solidus_subscriptions')
-rw-r--r--lib/solidus_subscriptions/configuration.rb2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/solidus_subscriptions/configuration.rb b/lib/solidus_subscriptions/configuration.rb
index f09740b..52c8fd4 100644
--- a/lib/solidus_subscriptions/configuration.rb
+++ b/lib/solidus_subscriptions/configuration.rb
@@ -43,7 +43,7 @@ module SolidusSubscriptions
end
def minimum_cancellation_notice
- @minimum_cancellation_notice ||= 1.day
+ @minimum_cancellation_notice ||= 0.days
end
def processing_queue