Skip to content
This repository has been archived by the owner on Feb 15, 2020. It is now read-only.

Fix string value deserialization, other improvements #15

Merged
merged 21 commits into from
Feb 15, 2020
Merged

Conversation

rpkilby
Copy link
Owner

@rpkilby rpkilby commented Feb 14, 2020

Contains a number of small improvements, but most importantly fixes #11 and fixes #9.

@codecov
Copy link

codecov bot commented Feb 14, 2020

Codecov Report

Merging #15 into master will increase coverage by 15.08%.
The diff coverage is 96.42%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #15       +/-   ##
===========================================
+ Coverage   80.95%   96.03%   +15.08%     
===========================================
  Files           2        3        +1     
  Lines          84      101       +17     
  Branches       14       14               
===========================================
+ Hits           68       97       +29     
+ Misses          9        1        -8     
+ Partials        7        3        -4
Impacted Files Coverage Δ
jsonfield/json.py 100% <100%> (ø)
jsonfield/forms.py 100% <100%> (+30.43%) ⬆️
jsonfield/fields.py 93.1% <88.88%> (+7.85%) ⬆️

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 388dcfb...15472ba. Read the comment docs.

Default help_text values are non-standard, and it should probably be
removed. However, moving this to the form field allows subclasses to
override or otherwise disable this.
@rpkilby rpkilby merged commit f7d1224 into master Feb 15, 2020
@rpkilby rpkilby deleted the cleanup branch February 15, 2020 01:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clean method fails on arbitrary string as a valid JSON JSON not formatted correctly in Django admin
1 participant