Skip to content

Conversation

@oliverb123
Copy link
Contributor

Recent changes left no good way to force disable person processing if you were in an identified context - this allows that again, by respecting the property if it's passed as set.

@oliverb123 oliverb123 requested review from a team, daibhin and hpouillot July 1, 2025 11:31
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

Modifies person processing behavior in the Python client to respect explicitly set $process_person_profile properties, allowing users to force-disable person processing in identified contexts.

  • Prevents automatic override of $process_person_profile in client.py when the property is already set in event properties

1 file reviewed, no comments
Edit PR Review Bot Settings | Greptile

@oliverb123 oliverb123 merged commit 50ab10c into master Jul 1, 2025
7 checks passed
@oliverb123 oliverb123 deleted the err/allow-force-disabling-person-processing branch July 1, 2025 12:26
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