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

5505 sort not working on some reports #5550

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

fergie-nz
Copy link
Contributor

Fixes #5505

Adds sorting fix to standard reports

πŸ‘©πŸ»β€πŸ’» What does this PR do?

Sorting now ignores caps
Removes obsolete name sort from item-usage
Fixes stock detail sort which didn't include wasm functions

πŸ’Œ Any notes for the reviewer?

πŸ§ͺ Testing

  • test any sorting in the item usage report
  • In any report see that sorting ignores caps

πŸ“ƒ Documentation

  • Part of an epic: documentation will be completed for the feature as a whole

@github-actions github-actions bot added this to the V2.4.0-RC4 milestone Nov 25, 2024
@github-actions github-actions bot added bug Something is borken needs triage feature: reports needs daily triage New bug to be triaged by the release team labels Nov 25, 2024
@roxy-dao roxy-dao removed needs triage needs daily triage New bug to be triaged by the release team labels Nov 25, 2024
@fergie-nz fergie-nz linked an issue Nov 26, 2024 that may be closed by this pull request
@roxy-dao roxy-dao changed the base branch from v2.4.0 to develop November 28, 2024 22:47
@roxy-dao roxy-dao modified the milestones: V2.4.0-RC6, v2.5.0 Nov 28, 2024
@github-actions github-actions bot modified the milestones: v2.5.0, V2.4.0-RC6 Nov 28, 2024
@roxy-dao roxy-dao modified the milestones: V2.4.0-RC6, v2.5.0 Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is borken feature: reports
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sort doesn't work on some reports
2 participants