hyperlane-xyz / hyperlane-monorepo

The home for Hyperlane core contracts, sdk packages, and other infrastructure
https://hyperlane.xyz
Other
295 stars 312 forks source link

Warp Route UI Feedback #1579

Closed jmrossy closed 1 year ago

jmrossy commented 1 year ago

Just went through our guide here https://app.gitbook.com/o/Pl1c4GYUuMTxkQTcVCdO/s/STktvmwamt5GXWv1jqOM/~/changes/hGo97smDCadKNzRc2Ezl/developers/warp-routes/deploy-the-ui-for-your-warp-route and noticing a few things GitBook GitBook Was able to successfully send from collateral chain to remote chain noted how we don't show the tx hash of the transferRemote or link to the explorer the latter would be nice until we have a progress screen The customize guide should probably show you how to remove certain things like the blue prompt for forking, or the title i think it would be nice to show balance on both origin and destination chain still, i see how the balance of the destination chain is only available after you hit continue A potential bug is that after i sent the weth to alfajores and then flipped the direction, the UI did not show that i have a balance of Weth on alfajores even though i definitely did i was actually able to send the weth back despite that the loading indicator on the way back still said that it was an approve and transferRemote when in reality it was only a transferRemote again would be nice to have a link to the explorer here (until the progress screen comes)

jmrossy commented 1 year ago

Mostly done in https://github.com/hyperlane-xyz/hyperlane-warp-ui-template/pull/10 but still need progress bar improvements

jmrossy commented 1 year ago

Done with https://github.com/hyperlane-xyz/hyperlane-warp-ui-template/pull/10 and https://github.com/hyperlane-xyz/hyperlane-warp-ui-template/pull/11

With exception of progress bar which is a separate task: #1592 and #1322