drf-writable-nested icon indicating copy to clipboard operation
drf-writable-nested copied to clipboard

Bypass deletion if set to partial update

Open thibaut-pro opened this issue 5 years ago • 3 comments

Proposal to fix #78

thibaut-pro avatar May 20 '20 01:05 thibaut-pro

Codecov Report

Merging #112 into master will increase coverage by 0.01%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #112      +/-   ##
==========================================
+ Coverage   98.12%   98.13%   +0.01%     
==========================================
  Files           3        3              
  Lines         213      215       +2     
==========================================
+ Hits          209      211       +2     
  Misses          4        4              
Impacted Files Coverage Δ
drf_writable_nested/mixins.py 98.02% <100.00%> (+0.01%) :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 9dfc270...a0a9852. Read the comment docs.

codecov-commenter avatar May 20 '20 01:05 codecov-commenter

Hi! Thanks for the contribution. Could you please add tests for your use case?

ruscoder avatar May 26 '21 17:05 ruscoder

Hi! Don't find this impudent but would it be possible to use another serializer parameter (not partial) to make the solution more flexible and save the current logic (the partial parameter already used)?

MishaGubsky avatar Feb 11 '22 09:02 MishaGubsky