Skip to content
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

Improve multisig performance #3126

Merged
merged 3 commits into from
Jun 27, 2024

Conversation

CedrikNikita
Copy link
Collaborator

@CedrikNikita CedrikNikita commented Jun 25, 2024

fixes #3092

Idea of this PR is to split all the multisig vaults into array of fives. And only dry-run five vault information in parallel.

Copy link

@Liubov-crypto
Copy link
Collaborator

There is no issues in console, but Send / Propose Tx modal broken and if I'm going into acc details to check Tx or token details wallet will became black screen.

2024-06-25.12.02.09.mov
2024-06-25.12.03.20.mov
2024-06-25.12.04.09.mov

@CedrikNikita CedrikNikita force-pushed the feature/improve-multisig-performance branch from f09bd5c to bd0d912 Compare June 25, 2024 09:22
Copy link

@Liubov-crypto
Copy link
Collaborator

Issue with black screen is remaining:
https://github.com/superhero-com/superhero-wallet/assets/69896204/b69ef37e-bc30-4c88-8ce7-7e8c6517481f

Also no fiat value:
am

@CedrikNikita CedrikNikita force-pushed the feature/improve-multisig-performance branch from bd0d912 to 0eccd65 Compare June 25, 2024 10:45
@CedrikNikita CedrikNikita marked this pull request as ready for review June 25, 2024 10:49
Copy link

@Liubov-crypto
Copy link
Collaborator

LGTM

@CedrikNikita CedrikNikita changed the title Improve multisig performance fix: do not overload node with dry-runs Jun 26, 2024
src/composables/multisigAccounts.ts Outdated Show resolved Hide resolved
src/composables/multisigAccounts.ts Outdated Show resolved Hide resolved
src/composables/multisigAccounts.ts Outdated Show resolved Hide resolved
@CedrikNikita CedrikNikita force-pushed the feature/improve-multisig-performance branch from 0eccd65 to a3c13dd Compare June 27, 2024 08:29
Copy link

@CedrikNikita CedrikNikita force-pushed the feature/improve-multisig-performance branch from a3c13dd to 7fa405b Compare June 27, 2024 10:08
Copy link

@CedrikNikita CedrikNikita force-pushed the feature/improve-multisig-performance branch from 7fa405b to af537dc Compare June 27, 2024 10:42
Copy link

@CedrikNikita CedrikNikita force-pushed the feature/improve-multisig-performance branch from 4743dd5 to 6120f59 Compare June 27, 2024 11:52
Copy link

1 similar comment
Copy link

Copy link

@CedrikNikita CedrikNikita force-pushed the feature/improve-multisig-performance branch from 999ec29 to 260a033 Compare June 27, 2024 12:16
@CedrikNikita CedrikNikita changed the title fix: do not overload node with dry-runs Improve multisig performance Jun 27, 2024
Copy link

@CedrikNikita CedrikNikita merged commit 8ea081a into develop Jun 27, 2024
6 checks passed
@CedrikNikita CedrikNikita deleted the feature/improve-multisig-performance branch June 27, 2024 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Transaction can not be executed
3 participants