-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Webwallet transfer lamports instead of sols #2267
Comments
@mvines, planning to squeeze this into Beacons? |
Not in a big rush to do this, seems a little peripheral to the v0.12 release goals |
On second thought, we might consider Sols being purely a |
@garious, that is one of the biggest complaints about ethereum, that eth/wei are not native erc20’s. |
I think the path to resolving this issue is to just have a switch in the example-webwallet that allows the user to work in lamports or SOLs (eg, Celsius or Fahrenheit). But along the way it would be nice to see some conversion helper functions added into https://github.com/solana-labs/solana-web3.js to avoid having to have the example-webwallet client write its own conversion logic. |
@mvines the bigger issue is similar to the problem that Ethereum has. Account's lamports are in the kernel. User defined Tokens are in userspace. The wallet needs to be aware of the Account's lamports only when dealing with contract state management, but for general purpose tokens it should deal with the userspace representation of sols. We are lacking a general purpose token ABI to allow that. |
Right. But that bigger issue is well outside the bounds of this particular issue. So you suggest we just close this issue then? |
@mvines no, thats fine. WebWallet should display sols instead of lamports. kick the token can down the road :) |
https://github.com/solana-labs/example-webwallet is archived now |
Depends-on: #160, specifically the SDK API changes to support this
The text was updated successfully, but these errors were encountered: