Details
-
Type:
Improvement
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: 3.6.0-b1
-
Component/s: None
-
Labels:None
Description
When a service is unsuspended (Services::unsuspend), we should also set the scheduled cancellation date (date_canceled) to null.
Issue Links
- blocks
-
CORE-1758
Add plugin to automatically cancel services
-
- Closed
-
Activity
Why is this feature desirable?
It seems like it may not be in some cases, such as if an admin schedules a service to cancel at the end of its term, but somewhere along the line a missed payment causes it to get suspended, and after it's paid, it is unsuspended, and then does not have its scheduled cancellation set any longer.
Tyson Phillips (Inactive)
added a comment - Why is this feature desirable?
It seems like it may not be in some cases, such as if an admin schedules a service to cancel at the end of its term, but somewhere along the line a missed payment causes it to get suspended, and after it's paid, it is unsuspended, and then does not have its scheduled cancellation set any longer. 
Yeah, that's a possibility. But overwhelmingly when a suspend service has payment applied to it which pulls it out of suspension, the service isn't intended to be cancelled any longer.
The only possible issue I see is if someone creates a term that is set to cancel at end of term and the user never pays the initial invoice, or their payment is returned, etc., so the service is suspend. After suspension, if the user pays the service will be unsuspended and no longer scheduled for cancellation.
Perhaps we should raise a ticket for those to cases so we can add some sort of scheduled cancellation tracking and only unschedule on unsuspend if it meets X,Y,Z criteria?