Skip to content

Insert new categories in the right place if the categories are already sorted#5455

Merged
ThiefMaster merged 3 commits into
indico:masterfrom
tomasr8:sort-new-categories
Aug 18, 2022
Merged

Insert new categories in the right place if the categories are already sorted#5455
ThiefMaster merged 3 commits into
indico:masterfrom
tomasr8:sort-new-categories

Conversation

@tomasr8
Copy link
Copy Markdown
Member

@tomasr8 tomasr8 commented Aug 17, 2022

New categories are currently always appended to the end, even if the list has been sorted.
This change checks whether the categories are sorted (either ascending or descending) and inserts
the new category in the correct place.

Comment thread indico/modules/categories/controllers/management.py Outdated
Comment thread indico/modules/categories/controllers/management.py Outdated
Comment thread indico/modules/categories/controllers/management.py Outdated
Comment thread indico/modules/categories/controllers/management.py Outdated
Comment thread indico/web/client/js/utils/sort.js
@tomasr8 tomasr8 force-pushed the sort-new-categories branch 2 times, most recently from 7e1ffb0 to 5e8f63f Compare August 17, 2022 11:56
@tomasr8 tomasr8 force-pushed the sort-new-categories branch from 5e8f63f to b4d035f Compare August 18, 2022 11:47
@ThiefMaster ThiefMaster enabled auto-merge (squash) August 18, 2022 11:50
@ThiefMaster ThiefMaster merged commit 5cbf07f into indico:master Aug 18, 2022
@tomasr8 tomasr8 deleted the sort-new-categories branch February 8, 2023 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants