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: handle pricing & features better for addons in ingestion flow #15219

Merged
merged 6 commits into from
Apr 25, 2023

Conversation

raquelmsmith
Copy link
Member

@raquelmsmith raquelmsmith commented Apr 24, 2023

Problem

The ingestion flow didn't include addon pricing, but would still subscribe someone to the Group Analytics addon if they sub'd to Product Analytics.

Changes

Plan comparison modal on ingestion now shows pricing for the main product and its addons:

image

The pricing is wrong in the above screenshot - I'll look into that and fix in a PR in the billing service, this is just for the UI.

And the ingestion screen lists features out better:

image

👉 Stay up-to-date with PostHog coding conventions for a smoother review.

How did you test this code?

👀

@raquelmsmith raquelmsmith self-assigned this Apr 24, 2023
so that we don't have to deal with pricing for that right now
Few people self-serve for enterprise anyway
@raquelmsmith
Copy link
Member Author

There is a bit more work to do here as @charlottemcginn and I discussed this morning but since this is behind a FF (mostly) I'm going to merge it in.

@raquelmsmith raquelmsmith merged commit d824554 into master Apr 25, 2023
@raquelmsmith raquelmsmith deleted the feat/billing-ingestion-addons branch April 25, 2023 16:56
@raquelmsmith raquelmsmith restored the feat/billing-ingestion-addons branch April 25, 2023 17:06
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