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

Gnosis Chain update #516

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

Gnosis Chain update #516

wants to merge 2 commits into from

Conversation

wackerow
Copy link
Member

Recently xDai has been rebranded to Gnosis Chain. This implements this new naming scheme.

@pettinarip Would you mind double checking that these changes won't affect anything on the subgraph side?

With associated svg asset, and update to README and hardhat.config. Recently rebranded from xDai.
@wackerow wackerow requested a review from pettinarip March 20, 2022 01:16
Copy link
Member

@pettinarip pettinarip left a comment

Choose a reason for hiding this comment

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

LGTM @wackerow 🚀

Just commented there to update the rpc urls.

One more thing. We are trying to deprecate this repo (we probably should mention this somewhere or block the repo) in favor of the monorepo. So, if you have the time, please submit the PR into the monorepo, I think it would be useful.

@@ -42,7 +42,7 @@ const config: HardhatUserConfig = {
url: process.env.JSONRPC_HTTP_URL || 'http://127.0.0.1:8545',
accounts,
},
xdai: {
gnosis: {
url: process.env.JSONRPC_HTTP_URL || 'https://rpc.xdaichain.com',
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
url: process.env.JSONRPC_HTTP_URL || 'https://rpc.xdaichain.com',
url: process.env.JSONRPC_HTTP_URL || 'https://rpc.gnosischain.com',

explorerLabel: 'Blockscout',
rpcUrl: 'https://rpc.xdaichain.com/',
rpcUrl: 'https://rpc.xdaichain.com',
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
rpcUrl: 'https://rpc.xdaichain.com',
rpcUrl: 'https://rpc.gnosischain.com',

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.

2 participants