search

caktus / django-treenav

Extensible, hierarchical, and pluggable navigation system for Django sites
BSD 3-Clause "New" or "Revised" License
130 stars 34 forks source link

Adds a "Rebuild Tree" button for MenuItem admin. #43

Closed jMyles closed 9 years ago

jMyles commented 9 years ago

Provides a workable end-around for #42.

Not pretty, but works to overcome the problem described by #42, in which the ordering becomes incongruent with the "order" attribute.

Patch funded by Kelly Creative.

mlavin commented 9 years ago

Thanks @jMyles! This seems like a reasonable work-around to me. @vkurup do you have any thoughts?

vkurup commented 9 years ago

Have to admit that I haven't wrapped my head around the actual underlying problem, but if this works, then I think it's completely reasonable.

@jMyles: We can merge this and make a release, unless you were working on other changes to address this issue.

jMyles commented 9 years ago

Hello Vinod. Hope you are well. :-)

Sadly, I too have declined to wrap my head around the actual underlying problem. Hopefully the django-mptt folk have a look at some point and our description here is able to illuminate something for them that we aren't quite seeing.

At this moment, we are not working on any additional fixes; the current patch is good enough.

vkurup commented 9 years ago

Thanks, hope you're well too! Appreciate your help, as well.

kkellydesign commented 9 years ago

You guys are awesome – I appreciate your work!