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

WebHost: Fixes multi-tracker checks sorting. #1893

Merged

Conversation

korydondzila
Copy link
Contributor

What is this fixing or adding?

Checks sorting should sort by checks completed not the formatted string displayed.
Uses orthogonal data

How was this tested?

Using dummy data for a generated game in multi-tracker

If this makes graphical changes, please attach screenshots.

Before After
Before After

@korydondzila korydondzila changed the title Fixes multi-tracker checks sorting. WebHost: Fixes multi-tracker checks sorting. Jun 25, 2023
@ThePhar ThePhar added is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. affects: webhost Issues/PRs that touch webhost and may need additional validation. labels Jun 25, 2023
@korydondzila korydondzila force-pushed the fix-multitracker-check-sorting branch from 1d3b20e to 9256e04 Compare June 26, 2023 22:58
@korydondzila korydondzila force-pushed the fix-multitracker-check-sorting branch from 9256e04 to cce8565 Compare June 26, 2023 23:08
@ThePhar ThePhar merged commit 76cace7 into ArchipelagoMW:main Jun 28, 2023
@korydondzila korydondzila deleted the fix-multitracker-check-sorting branch June 28, 2023 01:40
FlySniper pushed a commit to FlySniper/Archipelago that referenced this pull request Nov 14, 2023
Jouramie pushed a commit to Jouramie/Archipelago that referenced this pull request Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects: webhost Issues/PRs that touch webhost and may need additional validation. is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants