-
Notifications
You must be signed in to change notification settings - Fork 18
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
fix(tooling-dashboard): improve stake wizard #4454
fix(tooling-dashboard): improve stake wizard #4454
Conversation
This pull request has been deployed to Vercel. Latest commit: 2725997 ✅ Preview: https://iota-rebased-explorer-092603f1de8088243ccb8b861b23e-i3xpqtyyp.vercel.app |
This pull request has been deployed to Vercel. Latest commit: 2725997 ✅ Preview: https://apps-ui-200wfqh1m-iota1.vercel.app |
This pull request has been deployed to Vercel. Latest commit: 2725997 ✅ Preview: https://apps-backend-29a719c9ccbdb9b80c97b599a599d8953354da-gkfi4x79g.vercel.app |
This pull request has been deployed to Vercel. Latest commit: 2725997 ✅ Preview: https://wallet-dashboard-a225fcfa1e2f852205a61a7cd1d5d93e92-aufp45zpu.vercel.app |
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.
Another thing that I don't know if it's relevant to this PR, is that all setView
have ?.
, but the prop is not optional
we could update it here, wdyt?
apps/wallet-dashboard/components/Dialogs/Staking/StakeDialog.tsx
Outdated
Show resolved
Hide resolved
This pull request has been deployed to Vercel. Latest commit: d4f1476 ✅ Preview: https://apps-ui-dku29gyb9-iota1.vercel.app |
This pull request has been deployed to Vercel. Latest commit: d4f1476 ✅ Preview: https://apps-backend-29a719c9ccbdb9b80c97b599a599d8953354da-d6y8wlaeu.vercel.app |
This pull request has been deployed to Vercel. Latest commit: d4f1476 ✅ Preview: https://iota-rebased-explorer-092603f1de8088243ccb8b861b23e-7tmicbjhm.vercel.app |
This pull request has been deployed to Vercel. Latest commit: d4f1476 ✅ Preview: https://wallet-dashboard-a225fcfa1e2f852205a61a7cd1d5d93e92-h4p0bla4b.vercel.app |
This pull request has been deployed to Vercel. Latest commit: f788ed1 ✅ Preview: https://apps-ui-4263vmmjv-iota1.vercel.app |
This pull request has been deployed to Vercel. Latest commit: f788ed1 ✅ Preview: https://apps-backend-29a719c9ccbdb9b80c97b599a599d8953354da-jjc6w157c.vercel.app |
This pull request has been deployed to Vercel. Latest commit: f788ed1 ✅ Preview: https://iota-rebased-explorer-092603f1de8088243ccb8b861b23e-wkvoj8qkk.vercel.app |
This pull request has been deployed to Vercel. Latest commit: f788ed1 ✅ Preview: https://wallet-dashboard-a225fcfa1e2f852205a61a7cd1d5d93e92-r15d55yxk.vercel.app |
…ling-dashboard/improve-stake-wizzard
This pull request has been deployed to Vercel. Latest commit: 5d67367 ✅ Preview: https://apps-ui-nglo810di-iota1.vercel.app |
This pull request has been deployed to Vercel. Latest commit: 5d67367 ✅ Preview: https://iota-rebased-explorer-092603f1de8088243ccb8b861b23e-g2ksnyirk.vercel.app |
This pull request has been deployed to Vercel. Latest commit: 5d67367 ✅ Preview: https://apps-backend-29a719c9ccbdb9b80c97b599a599d8953354da-1dqes190s.vercel.app |
This pull request has been deployed to Vercel. Latest commit: 5d67367 ✅ Preview: https://wallet-dashboard-a225fcfa1e2f852205a61a7cd1d5d93e92-2tzt80k80.vercel.app |
21104d3
into
tooling-dashboard/add-timelocked-unstake-flow
* feat(dashboard): add unstake timelocked objects view * refactor: improve function * feat: add more unstaking information and rename constant * refactor: imports * fix: imports * fix: remove duplication collapsible * refactor: include both single stake and grouped in a unstake hook * refactor: unify usntake panel * fix: go back correctly to previous screen in staking * refactor: cleanup * refactor: remove popups * refactor: remove popups * revert "refactor: remove popups" * refactor: remove only unnecessary popups * refactor: divide hooks and move deeper inside components * fix: resolve re rendering issue * feat: remove redundant code * fix: update conditionally showing the stake dialog * fix(tooling-dashboard): improve stake wizard (#4454) * fix: improve stake wizzard * fix: add set view helper function * fix: add check for stake details --------- Co-authored-by: evavirseda <evirseda@boxfish.studio> * refactor: imports / exports * feat: add wait for transaction and refresh * feat: improve dialogs * feat(dashboard): minor fixes * fix: query key for timelocked staking * fix: revert constants changes * fix: disable stake button if no available amount for staking * fix: query key for timelocked staking transaction * fix: bigint serialization * fix: remove or check in the useNewStakeTimelockedTransaction --------- Co-authored-by: Bran <52735957+brancoder@users.noreply.github.com> Co-authored-by: evavirseda <evirseda@boxfish.studio> Co-authored-by: cpl121 <100352899+cpl121@users.noreply.github.com> Co-authored-by: cpl121 <cpeon@boxfish.studio> Co-authored-by: Branko <brankobosnic1@gmail.com>
* feat(dashboard): add unstake timelocked objects view * refactor: improve function * feat: add more unstaking information and rename constant * refactor: imports * fix: imports * fix: remove duplication collapsible * refactor: include both single stake and grouped in a unstake hook * refactor: unify usntake panel * fix: go back correctly to previous screen in staking * refactor: cleanup * refactor: remove popups * refactor: remove popups * revert "refactor: remove popups" * refactor: remove only unnecessary popups * refactor: divide hooks and move deeper inside components * fix: resolve re rendering issue * feat: remove redundant code * fix: add enter amount dialog component * fix: update conditionally showing the stake dialog * fix(tooling-dashboard): improve stake wizard (#4454) * fix: improve stake wizzard * fix: add set view helper function * fix: add check for stake details --------- Co-authored-by: evavirseda <evirseda@boxfish.studio> * refactor: imports / exports * feat: add wait for transaction and refresh * feat: improve dialogs * feat(dashboard): minor fixes * fix: query key for timelocked staking * fix: revert constants changes * fix: disable stake button if no available amount for staking * fix: add infoMessage var * fix: revert pnpm lock changes --------- Co-authored-by: JCNoguera <88061365+VmMad@users.noreply.github.com> Co-authored-by: evavirseda <evirseda@boxfish.studio> Co-authored-by: cpl121 <100352899+cpl121@users.noreply.github.com> Co-authored-by: cpl121 <cpeon@boxfish.studio>
* feat(dashboard): add unstake timelocked objects view * refactor: improve function * feat: add more unstaking information and rename constant * refactor: imports * fix: imports * fix: remove duplication collapsible * refactor: include both single stake and grouped in a unstake hook * refactor: unify usntake panel * fix: go back correctly to previous screen in staking * refactor: cleanup * refactor: remove popups * refactor: remove popups * revert "refactor: remove popups" * refactor: remove only unnecessary popups * refactor: divide hooks and move deeper inside components * fix: resolve re rendering issue * feat: remove redundant code * fix: update conditionally showing the stake dialog * fix(tooling-dashboard): improve stake wizard (#4454) * fix: improve stake wizzard * fix: add set view helper function * fix: add check for stake details --------- Co-authored-by: evavirseda <evirseda@boxfish.studio> * refactor: imports / exports * feat: add wait for transaction and refresh * feat: improve dialogs * feat(dashboard): minor fixes * fix: query key for timelocked staking * fix: revert constants changes * fix: disable stake button if no available amount for staking * fix: query key for timelocked staking transaction * fix: bigint serialization * fix: remove or check in the useNewStakeTimelockedTransaction --------- Co-authored-by: Bran <52735957+brancoder@users.noreply.github.com> Co-authored-by: evavirseda <evirseda@boxfish.studio> Co-authored-by: cpl121 <100352899+cpl121@users.noreply.github.com> Co-authored-by: cpl121 <cpeon@boxfish.studio> Co-authored-by: Branko <brankobosnic1@gmail.com>
* feat(dashboard): add unstake timelocked objects view * refactor: improve function * feat: add more unstaking information and rename constant * refactor: imports * fix: imports * fix: remove duplication collapsible * refactor: include both single stake and grouped in a unstake hook * refactor: unify usntake panel * fix: go back correctly to previous screen in staking * refactor: cleanup * refactor: remove popups * refactor: remove popups * revert "refactor: remove popups" * refactor: remove only unnecessary popups * refactor: divide hooks and move deeper inside components * fix: resolve re rendering issue * feat: remove redundant code * fix: add enter amount dialog component * fix: update conditionally showing the stake dialog * fix(tooling-dashboard): improve stake wizard (#4454) * fix: improve stake wizzard * fix: add set view helper function * fix: add check for stake details --------- Co-authored-by: evavirseda <evirseda@boxfish.studio> * refactor: imports / exports * feat: add wait for transaction and refresh * feat: improve dialogs * feat(dashboard): minor fixes * fix: query key for timelocked staking * fix: revert constants changes * fix: disable stake button if no available amount for staking * fix: add infoMessage var * fix: revert pnpm lock changes --------- Co-authored-by: JCNoguera <88061365+VmMad@users.noreply.github.com> Co-authored-by: evavirseda <evirseda@boxfish.studio> Co-authored-by: cpl121 <100352899+cpl121@users.noreply.github.com> Co-authored-by: cpl121 <cpeon@boxfish.studio>
* feat(dashboard): add unstake timelocked objects view * refactor: improve function * feat: add more unstaking information and rename constant * refactor: imports * fix: imports * fix: remove duplication collapsible * refactor: include both single stake and grouped in a unstake hook * refactor: unify usntake panel * fix: go back correctly to previous screen in staking * refactor: cleanup * refactor: remove popups * refactor: remove popups * revert "refactor: remove popups" * refactor: remove only unnecessary popups * refactor: divide hooks and move deeper inside components * fix: resolve re rendering issue * feat: remove redundant code * fix: update conditionally showing the stake dialog * fix(tooling-dashboard): improve stake wizard (#4454) * fix: improve stake wizzard * fix: add set view helper function * fix: add check for stake details --------- Co-authored-by: evavirseda <evirseda@boxfish.studio> * refactor: imports / exports * feat: add wait for transaction and refresh * feat: improve dialogs * feat(dashboard): minor fixes * fix: query key for timelocked staking * fix: revert constants changes * fix: disable stake button if no available amount for staking * fix: query key for timelocked staking transaction * fix: bigint serialization * fix: remove or check in the useNewStakeTimelockedTransaction --------- Co-authored-by: Bran <52735957+brancoder@users.noreply.github.com> Co-authored-by: evavirseda <evirseda@boxfish.studio> Co-authored-by: cpl121 <100352899+cpl121@users.noreply.github.com> Co-authored-by: cpl121 <cpeon@boxfish.studio> Co-authored-by: Branko <brankobosnic1@gmail.com>
Description of change
resolves #4308