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

Fix Evmos Public RPC Listings #3598

Merged
merged 12 commits into from
Oct 19, 2023
Merged

Conversation

KagemniKarimu
Copy link
Contributor

Lava in partnership with the Evmos team was tasked with providing incentivized public RPC endpoints as the default RPC for Evmos ecosystem. This PR adds ipRPC endpoints to rpc lists on both eip155-9000.json (Evmos Testnet) and eip155-9001.json (Evmos Mainnet). This PR is a potential pre-requisite for upstream changes on chainlist.org.

Copy link
Member

@ligi ligi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add your new RPCs at the end of the list

@KagemniKarimu
Copy link
Contributor Author

Please add your new RPCs at the end of the list

Does it matter that Evmos team has requested that Lava be the default RPC for their chains? Will putting the links at the end as opposed to beginning of the array change their order of priority?

@ligi
Copy link
Member

ligi commented Sep 29, 2023

Do you have any link to that claim?

@danburck
Copy link

danburck commented Oct 3, 2023

@ligi I'm Dan CPO at Altiplanic, the core contributor to the Evmos chain. The lava endpoints are delivering decentralized reliable RPC endpoints and should replace the existing default, due to their instability. This project also has been supported through the Encode Grants Programm on Evmos.

We kindly ask to support them as the new default.

@danburck
Copy link

@ligi @Ranahayat1122 is there anything blocking this? We have many users who suffer with the current default endpoint from Blockdaemon as it is not reliable.

Would appreciate, if we could merge this today.

Copy link
Member

@ligi ligi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please run the prettier to format your json according to the style defined here
e.g. run npx prettier --write _data/*/*.json

@KagemniKarimu
Copy link
Contributor Author

KagemniKarimu commented Oct 16, 2023

Please run the prettier to format your json according to the style defined here e.g. run npx prettier --write _data/*/*.json

I ran the linter and committed here. We should be good to go!

@danburck
Copy link

danburck commented Oct 17, 2023

@ligi We'd really if we could speed this up and get the PR merged. It's been three weeks and our users are still running into the same issue.

Here's a whole article about it https://twitter.com/EvmosDAO/status/1714386898674987236

@KagemniKarimu
Copy link
Contributor Author

Please run the prettier to format your json according to the style defined here e.g. run npx prettier --write _data/*/*.json

Please let me know if there's anything else to do to expedite this request! 💪🏿

@ligi ligi merged commit d74e3e8 into ethereum-lists:master Oct 19, 2023
4 checks passed
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.

3 participants