search

PostHog / posthog

🦔 PostHog provides open-source product analytics, session recording, feature flagging and A/B testing that you can self-host.
https://posthog.com
Other
19.39k stars 1.13k forks source link

fix(plugin-server): use tx for create person step of merge #23372

Closed bretthoerner closed 3 days ago

bretthoerner commented 3 days ago

Problem

createPerson in the merge wasn't using the outer txn.

Changes

Use same txn as the personless Distinct ID writes.

👉 Stay up-to-date with PostHog coding conventions for a smoother review.

Does this work well for both Cloud and self-hosted?

Yes

How did you test this code?

Existing, more to come