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.

Sign up for GitHub

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

Jump to bottom

fix: weird item sorting by `idx` #28721

Merged

ankush merged 1 commit into frappe:develop from ankush:item_sorting

Dec 3, 2021

Member

ankush commented Dec 3, 2021 •

edited

Weird edge case when idx is present those items appear on top.

Long term potential solution: Don't add these fields in non-table doctypes 😐

To test (side effects):
Not easy to reproduce the behaviour but:

Item list view should work as expected and show item sorted by last modified
Item filters should work as expected.


          fix: weird item sorting by idx

022c09f

ankush requested review from marination and rohitwaghchaure as code owners

December 3, 2021 13:34

ankush added old-backport version-12-hotfix backport version-13-hotfix and removed old-backport version-12-hotfix labels

ankush merged commit ba5a7ff into frappe:develop

mergify bot mentioned this pull request

fix: weird item sorting by idx (backport #28721) #28722

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment