-
Notifications
You must be signed in to change notification settings - Fork 42
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
Sort validators by validator status property #757
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #757 +/- ##
========================================
Coverage 88.02% 88.02%
========================================
Files 99 99
Lines 1628 1628
Branches 312 312
========================================
Hits 1433 1433
Misses 195 195
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
a90acce
to
2c10d11
Compare
2c10d11
to
f17e499
Compare
f17e499
to
6ac24f3
Compare
src/vendors/helpers.ts
Outdated
inactive: 2, | ||
} | ||
|
||
export const sortByStatus = (a, b) => ValidatorStatusPriority[a.status] - ValidatorStatusPriority[b.status] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm, our typescript allows parameters to not be typed? Will need to reconfigure that
6ac24f3
to
37ea17c
Compare
Closes #750
my preference here is to go with memoized selector, another option would be to use DataTable sortFunction prop. UX would be the same.
0