-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
fix(data-warehouse): Added an error message for empty data source files #21843
Conversation
Size Change: +650 B (0%) Total Size: 1.04 MB ℹ️ View Unchanged
|
📸 UI snapshots have been updated1 snapshot changes in total. 0 added, 1 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated3 snapshot changes in total. 0 added, 3 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated2 snapshot changes in total. 0 added, 2 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated1 snapshot changes in total. 0 added, 1 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated1 snapshot changes in total. 0 added, 1 modified, 0 deleted:
Triggered by this commit. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sweet
* master: (28 commits) fix: refactor update animation to do less (#21901) fix(plugin-server): add time component to person.force_upgrade (#21899) fix(dashboards): Indicate 404/400/500 properly (#21853) fix: recording exports (#21900) feat: Update SLAs, remove GitHub buttons (#21818) chore: Removed feature flag for heatmaps (#21886) fix(data-warehouse): Added an error message for empty data source files (#21843) feat(surveys): allow to stop surveys once reached enough responses (#21528) fix: patch missing pause method in rrweb (#21898) fix(hogql): make funnels fast again (#21890) fix: Explicitly call out readonly user fields (#21889) chore(dev): Allow mypy to run in VS Code extension (#21891) feat: Small heatmap UI fixes (#21849) feat: Improve Heatmaps UI and notices (#21887) fix: Redirection to keep query params (#21881) fix: hot path is cooler now (#21888) feat(data-warehouse): Moved the table model creation to the data pipeline activity (#21817) fix: snapshot response timings (#21885) chore(deps): Update posthog-js to 1.130.0 (#21884) feat(debug): tabs + better view for non-hogql-query nodes (#21867) ...
Problem
Changes
DatabaseTree
component where theonClick
would select the wrong row when the underlying data has been updatedDoes this work well for both Cloud and self-hosted?
How did you test this code?