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

feat: hook up pricing page to plans API #5181

Merged
merged 46 commits into from Feb 3, 2023

Conversation

raquelmsmith
Copy link
Member

@raquelmsmith raquelmsmith commented Jan 30, 2023

We now have a new plans API in the billing service that we can read from to get info about the plans we currently offer. We want to hook this up to the pricing page so that it's always up to date, and so that we can easily run experiments with our pricing plans, features, etc.

Changes

  • Sets up test and control versions for the pricing page
  • On the test pricing page it now reflects @corywatilo's overall design here: Proposed changes to plans, pricing page #5151
    • That design also includes some feature changes that aren't included in this PR, but may be experimented with in the very near future 🤓
    • Puts much more emphasis on the fact that we have an awesome free plan
  • On the control page it is largely the same but removes references to any self-hosted enterprise plan (since we plan to sunset the k8s version) and also obfuscates pricing for the enterprise plan (cause we want to talk to peeps)
  • Updates the Q&A at the bottom of the page
Before / Control After / Test
image image
image image
image image

We also now get a lil tooltip describing each feature:

image

Checklist

  • Titles are in sentence case
  • Feature names are in sentence case too
  • Words are spelled using American English
  • I have checked out our style guide

@corywatilo
Copy link
Collaborator

What is an "App metrics" integration?

image

@raquelmsmith
Copy link
Member Author

What is an "App metrics" integration?

Since Apps are an integration then it felt like it made sense to go here. But if you want it in one of the other feature groups then it can be moved fairly easily.

@raquelmsmith
Copy link
Member Author

@corywatilo this is good to go now. Tomorrow we will get an update out for the billing service that changes the free plan name to "Free." Otherwise can you give me a 👍 or 👎 on the rest of it?

Also excluding features from the table is easy so lmk if you think any of the features shouldn't be shown. I think there is a balance to strike between "make the free plan look awesome" vs "show that we do actually offer everything you need when you decide to upgrade." I think curating the features that are shown in this table is part of it.

* initial rebuild of plan matrix with flexbox

* most responsive styles in place

* hide some features, ui polish

* use prod billing service by default

* polish

* remove temp classes

* polished styles, removed old styles

* align all the things

* fix alignment

* icons, polish
@corywatilo
Copy link
Collaborator

corywatilo commented Feb 2, 2023

Copying over the todos that are beyond my abilities:

Todos

  • Change casing of section titles (to sentence case)

    image
  • Change "Product Analytics" to "Product analytics suite"

  • This "first" is likely coming from Stripe?

    image
  • Merge posthog/billing#115 along with this (sorry in advance for any merge issues there)

  • Only other unclear thing: "What happens if I exceed 1m events on the Free plan?"

@raquelmsmith
Copy link
Member Author

Only other unclear thing: "What happens if I exceed 1m events on the Free plan?"

Somebody kills a puppy.

(jk)

Not too concerned about this at the moment.

Copy link
Collaborator

@smallbrownbike smallbrownbike left a comment

Choose a reason for hiding this comment

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

Only blocker for me is the broken link in the job application confirmation modal! 👏

src/components/Job/Apply.tsx Outdated Show resolved Hide resolved
src/components/Pricing/AllPlans/index.tsx Show resolved Hide resolved
@raquelmsmith raquelmsmith merged commit 3190484 into master Feb 3, 2023
@raquelmsmith raquelmsmith deleted the fix/remove-enterprise-pricing branch February 3, 2023 19:38
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.

None yet

4 participants