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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants