search

django-polymorphic / django-polymorphic-tree

Polymorphic MPTT tree support for models
Other
170 stars 45 forks source link

Fix unresolved merge conflict in templates #58

Closed singleton11 closed 7 years ago

singleton11 commented 7 years ago

That option which has to be applied (by merged PR) didn't work, therefore I left the first option

codecov-io commented 7 years ago

Current coverage is 55.48% (diff: 100%)

Merging #58 into master will not change coverage

@@             master        #58   diff @@
==========================================
  Files            13         13          
  Lines           766        766          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits            425        425          
  Misses          341        341          
  Partials          0          0

Powered by Codecov. Last update 23af29a...e9636f2

vdboor commented 7 years ago

I've updated the README, thanks for highlighting that!

I'm not sure what you try to do by moving the @transaction.atomic inside the function as with transaction.atomic(). Is there any difference for you?