-
Notifications
You must be signed in to change notification settings - Fork 226
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
Behavioral targeting event store queries #5226
Behavioral targeting event store queries #5226
Conversation
77aa9d2
to
55b8983
Compare
Codecov ReportBase: 41.89% // Head: 42.69% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #5226 +/- ##
==========================================
+ Coverage 41.89% 42.69% +0.79%
==========================================
Files 173 174 +1
Lines 13267 13464 +197
==========================================
+ Hits 5558 5748 +190
- Misses 7709 7716 +7
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
Looking good!
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.
Drive by review: this is looking good. I have a few questions, but I won't stand in the way with another review cycle.
Great job!
…ehavior-targeting-event-store-queries
* add behavior event store query method * update test behavior to correctly use new methods * adjust query method and tests based on feedback
This PR adds a method to query into the event store. This will be used to find core active users, as well as users meeting other behavior criteria.
#5206 must be merged first.
EXP-2643
Pull Request checklist
[ci full]
to the PR title.Branch builds: add
[ac: android-components-branch-name]
and/or[fenix: fenix-branch-name]
to the PR title.