Skip to content

fix: ensure STX status updates across all networks#511

Closed
httpJunkie wants to merge 1 commit intomainfrom
fix/stx-status-stays-penging-on-chain-change
Closed

fix: ensure STX status updates across all networks#511
httpJunkie wants to merge 1 commit intomainfrom
fix/stx-status-stays-penging-on-chain-change

Conversation

@httpJunkie
Copy link
Copy Markdown
Contributor

Thanks for reviewing! Here's what you need to know about these changes:

Current State and Need for Change:

  • Currently, when users have a pending Smart Transaction (STX) and switch to a different network, the STX status fails to update
  • Users must switch back to the original network (Ethereum or BNB) to see status updates
  • This blocks users from initiating new transactions on other networks until they switch back
  • The root cause is that we're using the transaction's chainId instead of the storage key's chainId when fetching status updates

Solution:

  • Modified updateSmartTransactions to consistently use the storage key's chainId instead of the transaction's chainId
  • Added comprehensive tests to verify status updates work correctly across multiple networks
  • The fix ensures that STX status updates happen regardless of which network the user is currently on
  • Users can now freely switch networks without losing track of their pending transactions

Related Issues:

The changes are minimal but resolve a significant UX pain point where users were forced to switch back to original networks to see transaction status updates.

- STX status wasn't being updated when users switched
networks because we were using the transaction's chainId instead of the storage key chainId. This caused pending transactions to remain stuck until users switched back to the original network.

This fix ensures that smart transaction statuses are properly updated regardless of which network the user is currently on.
@httpJunkie httpJunkie added the bug Something isn't working label Apr 15, 2025
@httpJunkie httpJunkie self-assigned this Apr 15, 2025
@httpJunkie httpJunkie requested a review from a team April 15, 2025 12:19
@httpJunkie
Copy link
Copy Markdown
Contributor Author

The issue this fix was for seems to not be an issue anymore.

@httpJunkie httpJunkie closed this Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: STX status is not updated if user is on another network

1 participant