posthog

posthog copied to clipboard

Problem

Long running queries confuse users and they don't what's going on. The hedgehog running with no feedback gets frustrating.

Changes

Roll out a beta of a loading bar behind a feature flag

https://github.com/PostHog/posthog/assets/1855120/81fb8425-160d-4383-85a0-d1ecea0d6c30

How did you test this code?

Loading paths tested on local

📸 UI snapshots have been updated

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

• chromium: 0 added, 2 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.

📸 UI snapshots have been updated

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

• chromium: 0 added, 2 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.

📸 UI snapshots have been updated

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

• chromium: 0 added, 2 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.

Size Change: +97 B (+0.01%)

Total Size: 1.05 MB

ℹ️ View Unchanged

Filename	Size	Change
frontend/dist/toolbar.js	1.05 MB	+97 B (+0.01%)

_{compressed-size-action}

📸 UI snapshots have been updated

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

• chromium: 0 added, 2 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.

📸 UI snapshots have been updated

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

• chromium: 0 added, 2 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.

📸 UI snapshots have been updated

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

• chromium: 0 added, 2 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.

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ UnboundLocalError: local variable 'span' referenced before assignment posthog.tasks.tasks.process_query_task View Issue

_{Did you find this useful? React with a 👍 or 👎}