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
adds user contexts to userevents. #5361
adds user contexts to userevents. #5361
Conversation
e056dc9
to
8b739ff
Compare
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.
Needs an API bump, otherwise this is really nice and a clean solution to the problem, love it!
$from, | ||
$to, | ||
$joinsAndUserContext[0], | ||
$joinsAndUserContext[1], |
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.
I'm not loving the array by index here. It's completely self contained in this file, and probably easier to understand that I'm feeling, but reading it a few times it's hard to follow which of these is which. I'm mainly ruminating, I'm ok to keep it.
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.
it's gnarly.
8b739ff
to
22397d6
Compare
API version bumped ✔️ |
this is gnarly, but i can't think of a less invasive way to weave these data points in that doesn't involve substantial refactoring of the whole process.
22397d6
to
325044f
Compare
fixes #5338