-
Notifications
You must be signed in to change notification settings - Fork 154
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
Move Staking UI to Redux + SDK #1674
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Nice work! Just a few suggestions
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.
Nice work!
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.
Feedback see Discord
* enable all markets on optimism goerli (#1711) * Bump github/codeql-action from 2.1.32 to 2.1.35 (#1717) Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2.1.32 to 2.1.35. - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@4238421...b2a92eb) --- updated-dependencies: - dependency-name: github/codeql-action dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * minor improvements (typos and links) to README. (#1716) * use correct factory * Refactor isolated margin actions * order submission * fix disabled conditions * fix leverage input change * Move close isolated margin position to sdk and redux * Move Staking UI to Redux + SDK (#1674) * Some contracts and types * Add these * Maybe work? * It actually works * Remove rewards from getClaimableRewards return * Perf things * Fix small issues * Use vestEnabled * Remove anonymous functions in EscrowTable * Restructure SDK functions * Add approve default * Switch to using BigNumber amounts * Fix types * Handle disabling buttons when action is loading * Add multicall things * Remove unused useGetFiles hook * Make wei changes * Remove unused import * Replace ethers.BigNumber with BigNumber * Small performance improvement * More small optimizations * Remove redundant async annotation * Add consideration to README * Some fixes * Disable claim when totalRewards is zero * Fix unstake for LP tokens * Fix ABI issue * Claimable rewards * Refactor trade previews * start implementing delayed orders * fix trade previews and inputs * error handling * update fee info * set up delayed orders * cancel orders * execute orders * fix desired time and commit deposit * set up countdown timer * fix condition * Fix cross margin leverage * Improve cross margin preview speed * Refactor trade inputs * fix loader * fix fetching order * condition on gas estimate * fix some todos and defaults * fix volumes * remove additional funding rate query * open order condition * add buffer Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: leifu <leifu.chen@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Halvor Holsten Strand <halvor.holsten.strand@gmail.com> Co-authored-by: avclarke <adam@adamclarke.co> Co-authored-by: Oluwakorede Fashokun <koredefashokun@gmail.com>
Description
This PR moves the staking UI away from wagmi hooks and to using Redux with new SDK methods.
Related issue
N/A
Motivation and Context
N/A
How Has This Been Tested?
N/A
Screenshots (if appropriate):