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

Use cents instead of percentage to set discount for customer. #29934

Closed
wants to merge 1 commit into from

Conversation

amanagr
Copy link
Member

@amanagr amanagr commented May 3, 2024

Use #29957 for code reviews which is without stripe fixture data

First we need to run #29890 for exisitng customers (not required for testing this PR though)

cloud discount

Screenshot 2024-05-06 at 8 33 40 AM Screenshot 2024-05-06 at 8 32 59 AM Screenshot 2024-05-06 at 8 36 11 AM Screenshot 2024-05-06 at 8 40 01 AM Screenshot 2024-05-06 at 8 40 22 AM

basic free trial discount

Screenshot 2024-05-06 at 8 37 44 AM Screenshot 2024-05-06 at 8 39 27 AM

business discount

Screenshot 2024-05-06 at 8 40 53 AM Screenshot 2024-05-06 at 8 41 12 AM Screenshot 2024-05-06 at 8 41 52 AM Screenshot 2024-05-06 at 8 42 13 AM

next plan discount

Screenshot 2024-05-06 at 9 00 56 AM Screenshot 2024-05-06 at 8 48 53 AM Screenshot 2024-05-06 at 8 49 25 AM Screenshot 2024-05-06 at 9 05 06 AM

@zulip zulip deleted a comment from zulipbot May 6, 2024
@zulip zulip deleted a comment from zulipbot May 6, 2024
@zulip zulip deleted a comment from zulipbot May 6, 2024
@amanagr amanagr changed the title [WIP] Use cents instead of percentage to set discount for customer. Use cents instead of percentage to set discount for customer. May 6, 2024
This allows us to set the price of a plan exactly as discussed with
the customer.
@amanagr
Copy link
Member Author

amanagr commented May 6, 2024

Closing in favour of #29957 since we don't need the stripe fixtures as I used the same discount percentages in the tests which resulted in effectively no changes to generated fixtures.

@amanagr amanagr closed this May 6, 2024
@amanagr amanagr deleted the bill_bill_billing branch May 6, 2024 04:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants