vivianjeng / anon-transfer

Transfer X ETH with A account, withdraw Y ETH with B account.

Hi there 👋

anon-transfer's Issues

[fix] Only support `gwei` transfer

current ETH supply is 122 million ETH

unit	dec	bit
wei	122000000000000000000000000	87 bit
gwei	122000000000000000	57 bit
ETH	122000000	27 bit

If anon-transfer only supports gwei transfer, we can avoid bit constraints
#10

Switch network will cause fail to fetch

flow:

• log in
• it will automatically switch network with connect()
• then it executes userSignup()
• but it throw Fail to fetch when fetching wasm and zkey

Max deposit ETH

The maximum deposit and withdraw ETH is 2**64 wei ~ 18.44 ETH

https://github.com/Unirep/Unirep/blob/bfdef979eb7a47387591b96aa46b1b35121aa5a3/packages/circuits/circuits/reputation.circom#L133-L134

Overflow

Now reputation proof only supports 64-bit rep
but genProveReputationProof takes number as input

https://github.com/Unirep/Unirep/blob/bae7ee7f33eef45c42730e1b277ccaf138abd56d/packages/core/src/UserState.ts#L743

// number
> 2**64
? 18446744073709552000

// max safe integer
> 2**53-1
? 9007199254740991

// bigint
> BigInt(2)**BigInt(64)
? 18446744073709551616n

Fix1:

• use circuit template to input bigint string directly
• (Or write a custom circuit to store 253-bit data)

Fix2:

• frontend should alert the maximum withdraw amout

Complete mobile view

Complete mobile view

It can be better

Save latest data to indexed db / local storage

use the latest data to generate proof
See Unirep/Unirep#552

Switch to the correct network

Now users need to manually switch to sepolia, we can send a request for switch to the correct network.