Closed denisroldan closed 5 years ago
Merging #9 into master will increase coverage by
0.06%
. The diff coverage is100%
.
@@ Coverage Diff @@
## master #9 +/- ##
==========================================
+ Coverage 97.95% 98.01% +0.06%
==========================================
Files 5 5
Lines 98 101 +3
Branches 17 18 +1
==========================================
+ Hits 96 99 +3
Misses 2 2
Impacted Files | Coverage Δ | |
---|---|---|
django_bleach/models.py | 100% <100%> (ø) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 3944616...90cf839. Read the comment docs.
Thanks @denisroldan
Feature description
BleachField
now accepts nullable or blank values.Tested on
Python 3.6+ Django==1.11.20 bleach==3.1.0
Bug cause
As happened on #6, due to bleach changes here
clean
didn't acceptNone
values. That caused any model with a nullableBleachField
to raise an exception like this one: