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 visibility into transactions processing #34

Open
jmrossy opened this issue Jul 14, 2023 · 2 comments
Open

Improve visibility into transactions processing #34

jmrossy opened this issue Jul 14, 2023 · 2 comments
Assignees
Labels
UX end user experience warp-route

Comments

@jmrossy
Copy link
Collaborator

jmrossy commented Jul 14, 2023

State of Transaction

  • Current state of tx
  • Estimated time to completion, e.g. Stargate

Timeline Widget

Current behavior:
For most chains, the TransfersStatusModal shows a Timeline widget

Screenshot 2023-07-14 at 1 20 11 PM

Desired behavior:

Two improvements are needed:

  1. The individual segments of the timeline should clickable external links to the corresponding transaction on the target chain's block explorer (example tx page.
  2. The results from the useMessageTimeline hook (see Timeline in TransfersStatusModal) should be cached when the message is confirmed to be delivered so we don't re-request that data each time.

Item 1 will require changes to the timeline widget, which is in a separate repo here:
https://github.com/hyperlane-xyz/hyperlane-widgets - we may want to move this into monorepo

Use the multiProvider's method to get the explorer tx link.

@jmrossy
Copy link
Collaborator Author

jmrossy commented Jul 14, 2023

Let's also decrease the font size of the Sent/Finalized/etc. labels while we're making this change
Done in css, nevermind

@jmrossy
Copy link
Collaborator Author

jmrossy commented Aug 2, 2023

While fixing we should address this TODO as it relates to status caching mentioned in the description: https://github.com/hyperlane-xyz/hyperlane-warp-ui-template/blob/main/src/features/transfer/TransfersStatusModal.tsx#L142

@jmrossy jmrossy moved this from Next Sprint to Backlog in Hyperlane Tasks Jan 15, 2024
@avious00 avious00 changed the title Improve Timeline Widget UX Improve visibility into transactions processing Apr 18, 2024
@avious00 avious00 added the UX end user experience label Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UX end user experience warp-route
Projects
Status: Backlog
Development

No branches or pull requests

2 participants