-
Notifications
You must be signed in to change notification settings - Fork 1
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
Restore sorting feature on All Pools page #1679
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
6 Skipped Deployments
|
|
|
||
const PINNED_POOLS = [ | ||
// Pin the 182d Savings GYD pool to the top of the list | ||
// Remove this pinning on Wednesday, December 4, 2024 |
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.
Is this still relevant?
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.
Going to leave it pinned for now, since it has decent liquidity, but we should talk to folks and see if it makes sense to unpin.
* preventing the UI from freezing due to slow Alchemy responses. | ||
*/ | ||
export function usePoolsListIsFetching(): boolean { | ||
const totalRequests = useIsFetching(); |
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.
Do we need this hook? We could just do this in the component.
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.
good catch, this was unreferenced code.
Closes #1564
Brings back sorting.
makeQueryKey2
to standandarize query key lookupsusePoolsList
hook that returns sorted pool rowsHyperdrive---DeFi-Yield-your-way.7.webm