-
Notifications
You must be signed in to change notification settings - Fork 16
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
Feat: Multisig creation with remark #1800
base: tbaut-remove-matrix-3rd
Are you sure you want to change the base?
Conversation
Jest Unit tests resultsDuration: 68.689 seconds
Outcome: Passed | Total Tests: 740 | Passed: 739 | Failed: 0
|
…baut-create-multisig
There are still quite some things to iron out, such as error management for the form submission, UI glitches, and I'm investigating an issue I found in the multisig indexer, but I'd like to have your opinion on things such as:
|
src/renderer/widgets/CreateWallet/ui/MultisigWallet/ConfirmationStep.tsx
Outdated
Show resolved
Hide resolved
src/renderer/widgets/CreateWallet/ui/MultisigWallet/ConfirmationStep.tsx
Outdated
Show resolved
Hide resolved
src/renderer/widgets/CreateWallet/ui/MultisigWallet/FlexibleMultisigWallet.tsx
Outdated
Show resolved
Hide resolved
src/renderer/widgets/CreateWallet/ui/MultisigWallet/FlexibleMultisigWallet.tsx
Outdated
Show resolved
Hide resolved
src/renderer/widgets/CreateWallet/ui/MultisigWallet/MultisigWalletWizard.tsx
Outdated
Show resolved
Hide resolved
src/renderer/widgets/CreateWallet/ui/MultisigWallet/NameThresholdStep.tsx
Outdated
Show resolved
Hide resolved
src/renderer/widgets/CreateWallet/ui/MultisigWallet/SelectSignatoriesStep.tsx
Outdated
Show resolved
Hide resolved
@pgolovkin I addressed your comments in the last commit.
I need help, maybe @Asmadek @sokolova-an or @johnthecat can tell me how to do that. |
|
I've adapted the confirmation step to the new design. |
@Tbaut looks great!
Also please check with Sergy which popup has to be used for showing signatories on the confirmation screen. For example for staking it looks like on the screenshot |
src/renderer/entities/transaction/ui/MultisigCreationFees.tsx/MultisigCreationFees.tsx
Outdated
Show resolved
Hide resolved
To answer here, what I answered in tg: |
…a-spektr into feat/tbaut-create-multisig
I added the signer selection, with a "Back" button and without the token info for now. I need some more info to manage the multi-shard accounts. I pinged @johnthecat to have a chat. |
src/renderer/widgets/CreateWallet/ui/MultisigWallet/components/Signer.tsx
Outdated
Show resolved
Hide resolved
Now showing the Wallet rather than the accounts @pgolovkin in the signer selection. |
@Tbaut looks great! Thanks! |
closes #1756
closes #1807