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.

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

[Infrastructure] method name sortDescending should be renamed to sortAscending #4481

Closed
davetsay opened this issue Nov 18, 2021 · 1 comment
Closed

[Infrastructure] method name sortDescending should be renamed to sortAscending #4481

davetsay opened this issue Nov 18, 2021 · 1 comment
Assignees
@unlikelyzero
Labels
type:bug
Milestone
Target: 1.8.1

Comments

@davetsay
Copy link
Contributor

Summary

Method naming is confusing

Expected vs Current Behavior

A to Z is sort in ascending order so thats how it should be named.

Impact Check List

  • [] Data loss or misrepresented data?
  • [] Regression? Did this used to work or has it always been broken?
  • [] Is there a workaround available?
  • [] Does this impact a critical component?
  • [] Is this just a visual bug?

Steps to Reproduce

Internal change only. Can verify this ticket by verifying #4464

Environment

  • Open MCT Version:
  • Deployment Type:
  • OS:
  • Browser:

Additional Information
@davetsay davetsay mentioned this issue Nov 18, 2021
Merged
8 tasks
@davetsay davetsay added this to the Target: 1.8.1 milestone Nov 18, 2021
@shefalijoshi
Copy link
Contributor

Verified fixed.

@unlikelyzero unlikelyzero self-assigned this Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@unlikelyzero unlikelyzero

Labels
type:bug
Projects
None yet
Milestone
Target: 1.8.1
Development

No branches or pull requests
3 participants
@davetsay @shefalijoshi @unlikelyzero