search

tokamak-network / tokamak-bridge

An interface designed to bridge assets between Titan and Ethereum chains.
https://app.bridge.tokamak.network
6 stars 0 forks source link

[Cross Trade - Frontend & Backend bugs]: Provide Tokens for YOURSELF #192

Closed NegruGeorge closed 1 month ago

NegruGeorge commented 1 month ago

Can you specify where the bug exists?

https://dev-tokamak-bridge.vercel.app/pools In pools / pools modal

What happened?

I could Provide tokens on Sepolia with the same address that initiated the request on Titan Sepolia.

  1. go to provide.
  2. disconnect the wallet
  3. click provide on your own request
  4. connect wallet in the request modal
  5. You can provide for yourself now

Demo video on how it worked:

Screencast from 2024-09-11 23-10-28.webm

Notes: The Smart contract allow one user to provide for himself.

transaction that was made on sepolia: https://sepolia.etherscan.io//tx/0x6a833b49d4a27605e79c726ee77ddb382229ec6815fc038b532aa13f0704cce6

transaction on titan sepolia: https://explorer.titan-sepolia.tokamak.network/tx/0x800a3f841d415e8fc0d49526fccea6a5b506ad2a6dcd343a88ec4df33092ff25

What browsers are you seeing the problem on?

Google Chrome

Relevant log output

No response

Upload any screenshots to aid with explaining the bug.

video above

SonYoungsung commented 1 month ago

@monica-tokamak should we add a new design for this case or just pop up the edit modal or close?

SonYoungsung commented 1 month ago

We've decided to close the modal when this case comes up anywhere. Any other things will be automatically changed based on account change on SPA, so don't need to refresh it.

Thanks for your help!