Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update billing to include free-tier credit #1242

Merged
merged 1 commit into from
Oct 16, 2023
Merged

Conversation

jshearer
Copy link
Contributor

@jshearer jshearer commented Oct 16, 2023

Description:

The free-tier credit is intended to zero out usage incurred while the tenant is covered by the free tier, before their free trial starting. If they do not have a trial start date, the free tier credit covers all time.

Instead of implementing this as a credit capped at the free tier limits, we decided to built it as a blanket credit for all usage prior to the free trial start date. Reason being, the total due should still be $0 until the free trial is over. If we built it as a capped credit, tenants would be billed for their usage exceeding the free tier limits before they get moved into the free trial, which is not desired.

Screen Shot 2023-10-15 at 14 47 21

This change is Reviewable

The free-tier credit is intended to zero out usage incurred while the tenant is covered by the free tier, before their free trial starting. Instead of implementing this as a credit capped at the free tier limits, we decided to built it as a blanket credit for all usage prior to the free trial start date. Reason being, the total due should still be $0 until the free trial is over. If we built it as a capped credit, tenants would be billed for their usage exceeding the free tier limits before they get moved into the free trial, which is not desired.
Copy link
Member

@jgraettinger jgraettinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jgraettinger
Copy link
Member

Hold on merging for a moment, though. Having a discussion on "Free Tier" language.

@jgraettinger
Copy link
Member

Never mind, no changes & clear to merge. Thanks

@jshearer jshearer merged commit 000177d into master Oct 16, 2023
3 checks passed
@jgraettinger jgraettinger deleted the billing/free_tier_credit branch October 16, 2023 16:35
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