-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Store depth in the model #5
Comments
I agree. Would a PR help see this make its way into the source? |
@smebberson prs always help |
+1 |
It's a good idea. We are facing same problem here. This issue was created about 3.5 years ago. But no response yet :( |
Hello |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I think it would be better to store each node's level as a field in the model, so that we can generate the json representation of the tree easier and more efficiently. In large trees, it's not feasible to get level of each node. That's too many db calls.
The text was updated successfully, but these errors were encountered: