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

TS: Split _totalRetryAttempts parameter into separate variables for read and write transactions #653

Open
michalsmiarowski opened this issue Jul 12, 2023 · 0 comments

Comments

@michalsmiarowski
Copy link
Contributor

After a discussion on the pull request threshold-network/token-dashboard#537 (comment), it has been proposed to improve the user experience on the Threshold Dashboard by separating the _totalRetryAttempts parameter into separate variables for read and write transactions.

Currently, there is an issue where executing a transaction (such as a redemption request) and subsequently rejecting it with the wallet triggers the transaction to fire again after approximately 3 seconds. If the rejection occurs again, it appears as an endless loop. To identify the user rejection error, it requires rejecting the transaction four times. This situation is undesirable, and we aim to enhance the user experience.

To address this issue, we propose implementing separate retry variables for signing transactions and reading transactions. This separation will allow us to utilize the retry functionality effectively when reading data, while avoiding the repetitive transaction firing during rejection scenarios.

@michalsmiarowski michalsmiarowski changed the title TS: Split _totalRetryAttempts parameter into separate variables for reand and write transactions TS: Split _totalRetryAttempts parameter into separate variables for read and write transactions Jul 12, 2023
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

No branches or pull requests

1 participant