helix-bridge / wormhole-ui

https://wormhole-ui.vercel.app/
2 stars 1 forks source link

空投模块:必须成功连接Tron后才能进入表单页 #52

Closed 1022ranji closed 3 years ago

1022ranji commented 3 years ago

情况描述:点击“Connect To Tron”后没有判断用户是否有Tron的钱包,或者是否处于lock的状态,直接进入了表单页,并且在“Connect To Tron”一栏显示了“false”。 修改:

  1. 点击“Connect To Tron”后判断用户是否有Tron的钱包,当没有钱包时在右上角弹出提示“Please install Tronlink first.”(请先安装Tronlink)
  2. 当用户有Tronlink但是处于lock的状态时,提示“Please unlock Tronlink first.”(请解锁Tronlink) image image
sxlwar commented 3 years ago

why? 历史记录那块不是已经单独入口了,这个地方为什么又要耦合起来?

https://github.com/darwinia-network/wormhole-ui/blob/908408de396aa5dea3d0a7d28098f18954e5c250/src/utils/network/connection.tsx#L220

image

1022ranji commented 3 years ago

这个issue不是指空投记录页面,是下边空投点击进去的页面。 同时建议空投的历史记录仍然放在之前的位置,历史记录独立出去只显示跨链的记录。空投功能属于普通用户基本用不到的,所以尽量不明显,比如已经将“空投”模块放在右下角了,空投记录也可以从“历史记录”中分离出去,跟着空投页面即可。