Mirror
/
zulip
mirror of https://github.com/zulip/zulip.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

stripe: Fix manual licenses not being propagated to next plan. 

We already calculate the correct `billed_licenses` early in the
function, so just used that to fix the bug where a legacy server
scheduled for upgrade doesn't respect the manual license count
set by the user.
This commit is contained in:
Aman Agrawal 2023-12-12 18:22:51 +00:00 committed by Tim Abbott
parent 7587ee2289
commit ccaef86e20
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
corporate/lib/stripe.py
View File

@ -1180,6 +1180,7 @@ class BillingSession(ABC):
current_licenses_count = self.get_billable_licenses_for_customer(
customer, plan_tier, licenses
)
# In case user wants more licenses for the plan. (manual license management)
billed_licenses = max(current_licenses_count, licenses)
plan_params = {
"automanage_licenses": automanage_licenses,
@ -1240,9 +1241,7 @@ class BillingSession(ABC):
)
# Update license_at_next_renewal as per new plan.
assert last_ledger_entry is not None
last_ledger_entry.licenses_at_next_renewal = (
self.get_billable_licenses_for_customer(customer, plan_tier, licenses)
)
last_ledger_entry.licenses_at_next_renewal = billed_licenses
last_ledger_entry.save(update_fields=["licenses_at_next_renewal"])
remote_server_legacy_plan.status = CustomerPlan.SWITCH_PLAN_TIER_AT_PLAN_END
remote_server_legacy_plan.save(update_fields=["status"])