Skip to content
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

Async event mapping for all postgres instances #3627

Closed
wants to merge 10 commits into from

Conversation

mariusandra
Copy link
Collaborator

Changes

Checklist

  • All querysets/queries filter by Organization, by Team, and by User
  • Django backend tests
  • Jest frontend tests
  • Cypress end-to-end tests

@mariusandra mariusandra temporarily deployed to posthog-async-mapping-t-bjsayk March 11, 2021 13:30 Inactive
@timgl timgl temporarily deployed to posthog-async-mapping-t-zuvaql March 11, 2021 13:33 Inactive
@mariusandra mariusandra temporarily deployed to posthog-async-mapping-t-zuvaql March 11, 2021 14:17 Inactive
@mariusandra mariusandra temporarily deployed to posthog-async-mapping-t-zuvaql March 11, 2021 15:00 Inactive
@mariusandra
Copy link
Collaborator Author

I had to tweak many tests and add code to manually calculate the actions after events got added. Should be fine. The only problem I couldn't figure out was why the order of returned breakdown cohorts changed... but I think it's also fine... possibly?

Got to run, will leave it running to see what GitHub tests do

@mariusandra mariusandra temporarily deployed to posthog-async-mapping-t-zuvaql March 11, 2021 16:42 Inactive
@Twixes
Copy link
Member

Twixes commented Mar 17, 2021

Superseded by #3678.

@Twixes Twixes closed this Mar 17, 2021
@Twixes Twixes deleted the async-mapping-true-if-not-ee-2 branch March 17, 2021 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants