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

Updated in-app pricing page for CD #1273

Merged
merged 4 commits into from
Nov 13, 2023

Conversation

swoodward90
Copy link
Contributor

Summary

Labels

Test Plan

Checklist

  • If required, I have updated the Plural documentation accordingly.
  • I have added tests to cover my changes.
  • I have added a meaningful title and summary to convey the impact of this PR to a user.
  • I have added relevant labels to this PR to help with categorization for release notes.

@swoodward90 swoodward90 added frontend Changes related to the frontend enhancement New feature or request labels Nov 13, 2023
checked: true,
},
{
label: '1 cluster',
Copy link
Member

Choose a reason for hiding this comment

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

we can put this but the enforcement is going to be honor system on the cluster side

Copy link
Contributor

github-actions bot commented Nov 13, 2023

Visit the preview URL for this PR (updated for commit 20c0d51):

https://pluralsh--pr1273-swoodward-cd-pricing-ip7bmjx6.web.app

(expires Mon, 20 Nov 2023 20:43:00 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 16a5e8ed11dae87f435fa6b87804553fe417305e

@@ -174,6 +182,10 @@ function BillingPricingTable() {
<Div body2>Unlimited</Div>
<Div body2>${clusterMonthlyPricing}/cluster/month</Div>
<Div body2>${userMonthlyPricing}/user/month</Div>
<Div body2>$200/pack of 5 services/month</Div>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@michaeljguarino hard-coding this for now since I think we'll need the variable for service monthly pricing - TODO fix

Copy link
Member

Choose a reason for hiding this comment

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

that's ok yea it's not exposed in the api now (and it's not exactly a line-item due to weirdness w/ usage based billing as well)

@michaeljguarino michaeljguarino merged commit ac5e20c into master Nov 13, 2023
10 of 11 checks passed
@michaeljguarino michaeljguarino deleted the swoodward/cd-pricing-changes branch November 13, 2023 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request frontend Changes related to the frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants