Skip to content
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

Implement new EVM chains #637

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

nesitor
Copy link
Member

@nesitor nesitor commented Oct 31, 2024

Related Jira ticket : ALEPH-279

Self proofreading checklist

  • Is my code clear enough and well documented
  • Are my files well typed
  • New translations have been added or updated if new strings have been introduced in the frontend
  • Database migrations file are included
  • Are there enough tests
  • Documentation has been included (for new feature)

Changes

Changed the aleph_message dependency to use the last version with new EVM definitions.
Also created a class called EVMVerifier that gets the abstraction about how to verify EVM chains, and extend the EthereumVerifier to use it.

How to test

Try to send messages using that new EVM chains on the CHAIN field of the message, like "CHAIN": "POL".

Notes

This PR should be merged before at the deployment time.

@nesitor nesitor self-assigned this Oct 31, 2024
@nesitor nesitor force-pushed the andres-feature-implement_new_evm_chains branch 5 times, most recently from bdd37b6 to 0b3e221 Compare October 31, 2024 21:49
…ed that chains to the SignatureVerifier as EVM chains.
@nesitor nesitor force-pushed the andres-feature-implement_new_evm_chains branch from 0b3e221 to e585d2f Compare October 31, 2024 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant