-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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(dev): Show Celery cohort queries in CH debug modal #21173
Conversation
This comment was marked as off-topic.
This comment was marked as off-topic.
📸 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. |
Size Change: +71 B (0%) Total Size: 824 kB ℹ️ View Unchanged
|
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.
Seems useful 👍🏻
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.
Makes sense & looks good minus an annotation regarding a type issue.
> | ||
Refresh | ||
</LemonButton> | ||
</div> |
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.
Nice! I wanted to use the button quite often when there were no results yet.
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
Problem
Debugging an issue with cohorts, but I don't know what the actual query was, because it isn't attributed to the user who saved the cohort.
Changes
Cohort queries should now be attributed to the user, if it was indeed user-initiated.
How did you test this code?
Updated cohort API's
test_creating_update_and_calculating
.