-
Notifications
You must be signed in to change notification settings - Fork 28
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
#915 | abstract the network configuration for easiest multichain support #917
#915 | abstract the network configuration for easiest multichain support #917
Conversation
…15-abstract-the-network-configuration-for-easiest-multichain-support
✅ Deploy Preview for teloscan ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for testnet-teloscan ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for zkevm-testnet ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
'host': 'rpc.testnet.telos.net', | ||
'port': 443, | ||
'path': '/evm', | ||
}, | ||
'api': 'https://api-dev.telos.net/v1', | ||
'hyperion': 'https://rpc.testnet.telos.net', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zkjamesfish provided those new links in this PR he created:
#909
Fixes #915
Description
This PR refactorizes the configuration files to be clear the data needed and variable for each network.
It also adds a third network configuration temporally for zkEVM network. You need to run locally an indexer and wait for it to sync and then put up the API endpoint.
@zkjamesfish provided good links for the RPC endpoints in this other (not merged) PR:
#909