tbtc-v2 icon indicating copy to clipboard operation
tbtc-v2 copied to clipboard

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

Open michalsmiarowski opened this issue 2 years ago • 0 comments

After a discussion on the pull request https://github.com/threshold-network/token-dashboard/pull/537#discussion_r1259509819, 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 avatar Jul 12 '23 20:07 michalsmiarowski