Skip to content

Sync balance between substrate & pallet-vm #19

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
dm4 opened this issue May 28, 2021 · 5 comments
Closed

Sync balance between substrate & pallet-vm #19

dm4 opened this issue May 28, 2021 · 5 comments
Assignees
Labels
feature New feature or request

Comments

@dm4
Copy link

dm4 commented May 28, 2021

The substrate balance for substrate account address is not sync to the corresponding EVM account address at pallet-vm.

Current conversion from substrate account to EVM account:
https://github.com/ParaState/frontier/blob/ssvm/frame/vm/src/lib.rs#L169-L184

Current conversion from EVM account to substrate account:
https://github.com/ParaState/frontier/blob/ssvm/frame/vm/src/lib.rs#L200-L209

We need to find a way to sync balance between substrate & EVM accounts.

@dm4 dm4 added the feature New feature or request label May 28, 2021
@CaptainVincent
Copy link

這是目前 charge gas fee 的架構, ① ② 是有想到可能可以改進的地方
IconFundationEnhance

@yanganto
Copy link

The upstream VM frame is upgraded to frame v2, should we take this into consideration?
polkadot-evm#380

@CaptainVincent
Copy link

CaptainVincent commented May 31, 2021

paritytech#380

In progress on migrate branch.
still working on fix remain ts-tests fail cases.

@CaptainVincent
Copy link

Update: Merge official frontier/master latest version 6e2e908 into ssvm done in #20
(including paritytech#380)

@yanganto
Copy link

yanganto commented Jun 2, 2021

After discussion, the issue is invalid and there will be other issues to address the business problem.

#27 #28 #29

@yanganto yanganto closed this as completed Jun 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants