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

[CWM] Sort search results alphabetically #89

Open
ColmDC opened this issue Nov 22, 2024 · 2 comments
Open

[CWM] Sort search results alphabetically #89

ColmDC opened this issue Nov 22, 2024 · 2 comments

Comments

@ColmDC
Copy link

ColmDC commented Nov 22, 2024

Description
There is currently no descernable order to the search results.
Sort them alphabetically

Image

Acceptance Criteria
All search results appear in alphabetical order.

@rogup
Copy link
Contributor

rogup commented Nov 23, 2024

This isn't trivial, so will have to wait until after Delhi.

Alphabetic sorting by name needs to happen before or during the generation of the BE data. We can't sort on the front-end since data is paginated and we don't have the full set of names. The server can't do it when loading the data, since sorting data would change the order of indexes, which then wouldn't match the item data files. And we can't do a sort before we search every set of search results, since this would take a lot of computation power and impact performance.

I think the best time to do the sorting is as part of the CSV generation, which is @wu-lee's realm of knowledge. Maybe just with a simple csv sort command line script.

@ColmDC
Copy link
Author

ColmDC commented Nov 23, 2024

Grand. Moved into backlog.

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

No branches or pull requests

2 participants