-
-
Notifications
You must be signed in to change notification settings - Fork 490
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
Feat (History Events): Consistent free and premium #7923
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #7923 +/- ##
===========================================
+ Coverage 72.07% 72.14% +0.07%
===========================================
Files 1179 1182 +3
Lines 107927 108747 +820
Branches 12867 12873 +6
===========================================
+ Hits 77783 78460 +677
- Misses 28221 28364 +143
Partials 1923 1923
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
41d6415
to
88e500b
Compare
177f9d2
to
147a8bc
Compare
Signed-off-by: OjusWiZard <[email protected]>
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.
Code changes look good to me. If after this filtering works both for premium and for free and has been tested it's fine. I am not testing it myself though so am taking @OjusWiZard and @yabirgb word for it.
Frontend code looks fine, also tried the functionality in frontend and it works |
Failed test is a remote error. |
Closes #6115
Checklist
get_history_events()
to return consistent results with premium and free modes.