Skip to content
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

Order field in page API not working on root level #1354

Closed
timobrembeck opened this issue Apr 12, 2022 · 0 comments · Fixed by #1355
Closed

Order field in page API not working on root level #1354

timobrembeck opened this issue Apr 12, 2022 · 0 comments · Fixed by #1355
Assignees
Labels
🐛 bug Something isn't working ❗ prio: medium Should be scheduled in the forseeable future.
Milestone

Comments

@timobrembeck
Copy link
Member

Describe the Bug

We use the lft field of the MPTT model to convert them to order fields which is used by the app.
However, on the root level this does not work, because all root pages have the same lft values.

Steps to Reproduce

Access API and inspect root pages

Expected Behavior

The root pages should be ordered by their tree ids

Actual Behavior

All root pages have the order field 1 because it's their lft value

@timobrembeck timobrembeck added 🐛 bug Something isn't working ❗ prio: medium Should be scheduled in the forseeable future. labels Apr 12, 2022
@timobrembeck timobrembeck added this to the Version 1.1 milestone Apr 12, 2022
@timobrembeck timobrembeck self-assigned this Apr 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working ❗ prio: medium Should be scheduled in the forseeable future.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant