-
Notifications
You must be signed in to change notification settings - Fork 244
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
Filter numeric only fields in Widget queries #2686
Filter numeric only fields in Widget queries #2686
Conversation
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## dev #2686 +/- ##
==========================================
- Coverage 53.22% 52.63% -0.59%
==========================================
Files 119 122 +3
Lines 9924 10153 +229
==========================================
+ Hits 5282 5344 +62
- Misses 4642 4809 +167
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
f9b77c9
to
69f9571
Compare
@ziv17 Not sure if necessary for this pr, but note that in news_flash table, for some reason yishuv_symbol is null even when yishuv_name exist, I suggest changing this and making sure we insert yishuv_symbol is inserted to news_flash table (and perhaps backfilling it). |
currently there are only 3 injury severities
@ziv17 looks great! I |
cf978e0
to
2a5b5bb
Compare
@ziv17 thanks for the fix! merged :) |
Closing #2673