Skip to content
This repository has been archived by the owner on Mar 13, 2023. It is now read-only.

Handle dvm display issues #356

Merged
merged 10 commits into from
Nov 9, 2020
Merged

Handle dvm display issues #356

merged 10 commits into from
Nov 9, 2020

Conversation

boundless-forest
Copy link
Member

@boundless-forest boundless-forest commented Oct 27, 2020

Desc

Note: This PR based on #334

Review Suggestion

Two commits:

The first commit just replaces the pallet-evm path to facilitate testing, you can jump directly to the second.

To see the corresponding evm pallet change, please refer to This link

@boundless-forest boundless-forest self-assigned this Oct 27, 2020
@boundless-forest boundless-forest marked this pull request as draft October 27, 2020 09:18
@boundless-forest boundless-forest marked this pull request as ready for review October 27, 2020 09:18
@boundless-forest boundless-forest marked this pull request as draft October 30, 2020 09:16
@boundless-forest boundless-forest force-pushed the handle-decimal-dvm branch 3 times, most recently from e2e4170 to 310c15b Compare November 3, 2020 06:05
@aurexav
Copy link
Member

aurexav commented Nov 3, 2020

Ready for review?

If not remove the label.

@boundless-forest
Copy link
Member Author

Ready for review?

If not remove the label.

Wait minutes, I am working on fixing the conflict and rebase to the latest integrate-frontier-substrate-2.0 branch.

@aurexav
Copy link
Member

aurexav commented Nov 3, 2020

What's the substrate PR link?

@boundless-forest
Copy link
Member Author

What's the substrate PR link?

Please check this commit firstly: darwinia-network/substrate@50a2e0c.

If you all approve this pr modification, then, I will raise the substrate pull request.

Base automatically changed from integrate-frontier-substrate-2.0.0 to master November 3, 2020 10:36
@boundless-forest boundless-forest marked this pull request as ready for review November 3, 2020 10:58
frame/dvm/ethereum/src/account_basic.rs Outdated Show resolved Hide resolved
frame/dvm/ethereum/src/account_basic.rs Show resolved Hide resolved
frame/dvm/ethereum/src/lib.rs Outdated Show resolved Hide resolved
frame/dvm/ethereum/src/lib.rs Show resolved Hide resolved
frame/dvm/ethereum/src/account_basic.rs Show resolved Hide resolved
@boundless-forest boundless-forest marked this pull request as draft November 5, 2020 06:41
@boundless-forest boundless-forest marked this pull request as ready for review November 6, 2020 00:58
@hackfisher
Copy link
Contributor

Related commit changes to substrate: darwinia-network/substrate@50a2e0c

@boundless-forest
Copy link
Member Author

@hackfisher @AurevoirXavier Please take some time to check this PR again.

@hackfisher
Copy link
Contributor

@hackfisher @AurevoirXavier Please take some time to check this PR again.

@AurevoirXavier Please help review and merge this pull request, thanks.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants