search

django-polymorphic / django-polymorphic-tree

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

Update stylable_admin_list.py to fix models.FieldDoesNotExist exception #80

Closed vesnikos closed 2 years ago

vesnikos commented 3 years ago

a quick PL to fix models.FieldDoesNotExist exception that is happening with django 3.1:

FieldDoesNotExist should be imported from exceptions now, as it's no longer referenced at django.db.model anymore? I think it should retain compatibility with django2

codecov[bot] commented 3 years ago

Codecov Report

Merging #80 into master will not change coverage. The diff coverage is 33.33%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #80   +/-   ##
=======================================
  Coverage   64.96%   64.96%           
=======================================
  Files          12       12           
  Lines         762      762           
=======================================
  Hits          495      495           
  Misses        267      267
Impacted Files Coverage Δ
...lymorphic_tree/templatetags/stylable_admin_list.py 18.58% <33.33%> (ø)

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 17a125f...891af27. Read the comment docs.

vdboor commented 2 years ago

thanks, this was merged some time ago!