Skip to content
This repository has been archived by the owner on Apr 4, 2024. It is now read-only.

Allow evm to call native modules through logs #416

Closed
yihuang opened this issue Aug 9, 2021 · 0 comments · Fixed by #417
Closed

Allow evm to call native modules through logs #416

yihuang opened this issue Aug 9, 2021 · 0 comments · Fixed by #417

Comments

@yihuang
Copy link
Contributor

yihuang commented Aug 9, 2021

Proposal: Allow evm to call native modules through logs

#394

Current behavior: evm can't call native modules

Desired behavior: allow evm to call native modules

Use case:

  • manage native tokens
    • ibc tokens
    • gravity bridge
  • many potential
@yihuang yihuang mentioned this issue Aug 9, 2021
Merged
11 tasks
yihuang referenced this issue in yihuang/ethermint Sep 1, 2021
@yihuang
Allow evm to call native modules through logs
9f8f0dd 
Closes #416

comment

add txHash parameter

review suggestions

add hooks test
fedekunze added a commit that referenced this issue Sep 2, 2021
@yihuang @fedekunze
evm: implement ADR-002 EVM Hooks (#417)
089afe4 
* Allow evm to call native modules through logs

Closes #416

comment

add txHash parameter

review suggestions

add hooks test

* Update x/evm/keeper/hooks.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* Update x/evm/keeper/hooks_test.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* Update x/evm/keeper/keeper.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* Update x/evm/keeper/keeper.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* Update x/evm/keeper/keeper.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* use table tests

* update adr comment

* update adr

* changelog

* Update CHANGELOG.md

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow evm to call native modules through logs yihuang/ethermint
1 participant
@yihuang