Skip to content

refactor(billing): rename GraceWindowDays to GracePeriodDays - #1313

Merged
iammukeshm merged 1 commit into
fullstackhero:mainfrom
marcelo-maciel:refactor/grace-period-naming
Jun 24, 2026
Merged

refactor(billing): rename GraceWindowDays to GracePeriodDays#1313
iammukeshm merged 1 commit into
fullstackhero:mainfrom
marcelo-maciel:refactor/grace-period-naming

Conversation

@marcelo-maciel

Copy link
Copy Markdown
Contributor

What

The tenant billing grace span is named "grace window" in code (GraceWindowDays in both TenantBillingOptions and Identity's TenantGraceOptions, the Billing:GraceWindowDays config key, and the _graceWindowDays field) while the user-facing and conceptual references call it the grace period. "Grace period" is the standard SaaS billing term; this standardizes the vocabulary on it.

Changes

  • Rename GraceWindowDaysGracePeriodDays in both options classes, all references, and the _graceWindowDays field.
  • Rename the bound config key Billing:GraceWindowDaysBilling:GracePeriodDays.
  • Align the comments / XML docs that still said "grace window".

Breaking change (config)

Deployments that set Billing:GraceWindowDays must rename the key to Billing:GracePeriodDays. The default (7 days) is unchanged.

Notes

  • No public contract change: the GraceEndsUtc DTO/event fields are untouched.
  • Multitenancy (90/90) and Identity (312/312) unit tests pass; build is clean (TreatWarningsAsErrors).
  • A companion docs PR (changelog) will follow in fullstackhero/docs.
The tenant billing grace span is named "grace window" in code (GraceWindowDays
option in both TenantBillingOptions and Identity's TenantGraceOptions, the
Billing:GraceWindowDays config key, and the _graceWindowDays field) while every
user-facing and conceptual reference calls it the grace period. "Grace period"
is the standard SaaS billing term; standardize the vocabulary on it.

- Rename the GraceWindowDays property in both options classes, the bound config
  key (Billing:GraceWindowDays -> Billing:GracePeriodDays), all references, and
  the _graceWindowDays field to GracePeriodDays.
- Align the surrounding comments/XML docs that still said "grace window".

BREAKING (config): deployments setting Billing:GraceWindowDays must rename the
key to Billing:GracePeriodDays. The default (7 days) is unchanged. No public
contract change: the GraceEndsUtc DTO/event fields are untouched.
@iammukeshm

Copy link
Copy Markdown
Member

Thanks @marcelo-maciel — clean, consistent standardization on the conventional SaaS term. Just a reminder to land the companion changelog entry in the docs repo for the Billing:GraceWindowDaysBilling:GracePeriodDays config rename (breaking for anyone who set it). Merging. 🙏

@iammukeshm
iammukeshm merged commit f0e6481 into fullstackhero:main Jun 24, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants