posthog icon indicating copy to clipboard operation
posthog copied to clipboard

feat: Add warning that we will autopay open invoices to unsubscribe modal

Open xrdt opened this issue 9 months ago • 4 comments

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?

xrdt avatar Apr 29 '24 19:04 xrdt

📸 UI snapshots have been updated

5 snapshot changes in total. 0 added, 5 modified, 0 deleted:

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Apr 29 '24 19:04 posthog-bot

📸 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.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Apr 29 '24 19:04 posthog-bot

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%)

compressed-size-action

github-actions[bot] avatar Apr 29 '24 19:04 github-actions[bot]

📸 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.

👉 Review this PR's diff of snapshots.

posthog-bot avatar May 01 '24 05:05 posthog-bot

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:

Screenshot 2024-05-08 at 11 05 38 AM

Some random exception:

Screenshot 2024-05-08 at 11 21 50 AM

xrdt avatar May 08 '24 18:05 xrdt

📸 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.

👉 Review this PR's diff of snapshots.

posthog-bot avatar May 08 '24 18:05 posthog-bot