silentsokolov / django-admin-rangefilter

A Django app that lets you filter data by date range and numeric range in the admin UI
MIT License
721 stars 106 forks source link

NumericRangeFilter: Fix filter for the value zero #105

Closed autoantwort closed 1 year ago

autoantwort commented 1 year ago

The filtering currently does not work if the boundary is zero.

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (d1ad13b) 86.80% compared to head (c9ee240) 86.80%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #105 +/- ## ======================================= Coverage 86.80% 86.80% ======================================= Files 11 11 Lines 614 614 ======================================= Hits 533 533 Misses 81 81 ``` | [Files Changed](https://app.codecov.io/gh/silentsokolov/django-admin-rangefilter/pull/105?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Dmitriy+Sokolov) | Coverage Δ | | |---|---|---| | [rangefilter/filters.py](https://app.codecov.io/gh/silentsokolov/django-admin-rangefilter/pull/105?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Dmitriy+Sokolov#diff-cmFuZ2VmaWx0ZXIvZmlsdGVycy5weQ==) | `80.86% <100.00%> (ø)` | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

silentsokolov commented 1 year ago

Thanks you!