Details
-
Type:
Improvement
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: 4.6.0-b1
-
Component/s: None
-
Labels:None
Description
Currently we deactivate cWatch licenses in three cases:
- When there is an error during service creation
- When there is an error while adding licenses to a service
- When a service is canceled
In each of these places, our integration should remove any domains associated with the licenses before deactivating them. Our integration already has a method CwatchApi::removeSite() for just this purpose. If possible abstract the action for these three cases to keep things DRY.
Activity
| Field | Original Value | New Value |
|---|---|---|
| Rank | Ranked higher |
| Story Points | 2 |
| Sprint | 4.6.0 Sprint 3 [ 79 ] |
| Rank | Ranked higher |
| Remaining Estimate | 0 minutes [ 0 ] | |
| Time Spent | 2 hours [ 7200 ] | |
| Worklog Id | 12009 [ 12009 ] |
| Status | Open [ 1 ] | In Review [ 5 ] |
| Resolution | Fixed [ 1 ] |
| Time Spent | 2 hours [ 7200 ] | 2 hours, 35 minutes [ 9300 ] |
| Worklog Id | 12010 [ 12010 ] |
Tyson Phillips (Inactive)
made changes -
| Sprint | 4.6.0 Sprint 3 [ 79 ] | 4.6.0 Sprint 3, 4.6.0 Sprint 4 [ 79, 80 ] |
Tyson Phillips (Inactive)
made changes -
| Rank | Ranked higher |
Tyson Phillips (Inactive)
made changes -
| Description |
Currently we deactivate cWach licenses in three cases:
# When there is an error during service creation # When there is an error while adding licenses to a service # When a service is canceled In each of these places, our integration should remove any domains associated with the licenses before deactivating them. Our integration already has a method CwatchApi::removeSite() for just this purpose. If possible abstract the action for these three cases to keep things DRY. |
Currently we deactivate cWatch licenses in three cases:
# When there is an error during service creation # When there is an error while adding licenses to a service # When a service is canceled In each of these places, our integration should remove any domains associated with the licenses before deactivating them. Our integration already has a method CwatchApi::removeSite() for just this purpose. If possible abstract the action for these three cases to keep things DRY. |
Tyson Phillips (Inactive)
made changes -
| Time Spent | 2 hours, 35 minutes [ 9300 ] | 2 hours, 37 minutes [ 9420 ] |
| Worklog Id | 12050 [ 12050 ] |
You already approved it and I merged it
Tyson Phillips (Inactive)
made changes -
| Status | In Review [ 5 ] | Closed [ 6 ] |
I don't see a PR for this on github