posthog
posthog copied to clipboard
feat: Add warning that we will autopay open invoices to unsubscribe modal
Problem
Also change the bottom banner to a collapse
Ties in with billing changes: https://github.com/PostHog/billing/pull/573 https://github.com/PostHog/billing/pull/583
https://github.com/PostHog/posthog/assets/21014901/14167528-1ecb-42b7-a99e-c92a85ee293f
Changes
👉 Stay up-to-date with PostHog coding conventions for a smoother review.
Does this work well for both Cloud and self-hosted?
How did you test this code?
📸 UI snapshots have been updated
5 snapshot changes in total. 0 added, 5 modified, 0 deleted:
-
chromium
: 0 added, 5 modified, 0 deleted (diff for shard 1, diff for shard 2) -
webkit
: 0 added, 0 modified, 0 deleted
Triggered by this commit.
📸 UI snapshots have been updated
1 snapshot changes in total. 0 added, 1 modified, 0 deleted:
-
chromium
: 0 added, 1 modified, 0 deleted (diff for shard 2) -
webkit
: 0 added, 0 modified, 0 deleted
Triggered by this commit.
Size Change: -26 B (0%)
Total Size: 1.05 MB
ℹ️ View Unchanged
Filename | Size | Change |
---|---|---|
frontend/dist/toolbar.js |
1.05 MB | -26 B (0%) |
📸 UI snapshots have been updated
4 snapshot changes in total. 0 added, 4 modified, 0 deleted:
-
chromium
: 0 added, 4 modified, 0 deleted (diff for shard 1) -
webkit
: 0 added, 0 modified, 0 deleted
Triggered by this commit.
This is integrated both with https://github.com/PostHog/posthog/pull/22057 and https://github.com/PostHog/billing/pull/573
When there is no active payment method:
https://github.com/PostHog/posthog/assets/21014901/ec82a278-3e26-48f9-9a1f-b28c436f991e
When we can't pay the invoice for some reason:
https://github.com/PostHog/posthog/assets/21014901/a96728d1-6d6b-4dab-8c8d-5a77dded4831
When we can't pay the invoice for the subscription that just closed. This circumstance matters much less because we block subscriptions when there are open invoices. I'm honestly not even sure we should show this state as the unsubscription succeeded:
Some random exception:
📸 UI snapshots have been updated
4 snapshot changes in total. 0 added, 4 modified, 0 deleted:
-
chromium
: 0 added, 4 modified, 0 deleted (diff for shard 1) -
webkit
: 0 added, 0 modified, 0 deleted
Triggered by this commit.