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

Remove Goerli from codebase #1314

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

Remove Goerli from codebase #1314

wants to merge 3 commits into from

Conversation

kronosapiens
Copy link
Member

Closes #1219

@kronosapiens
Copy link
Member Author

kronosapiens commented Dec 12, 2024

@area do you have the syncFrom and colonyNetwork addresses for the rinkeby, ropsten, and kovan testnets? For the miner docs.

@area
Copy link
Member

area commented Dec 16, 2024

Are any of those still supported? I'm not sure if we ever ended up on ropsten or kovan.

@kronosapiens
Copy link
Member Author

kronosapiens commented Dec 16, 2024

Noting that we are on Arbitrum-Sepolia but not the other testnets, at 0x7777494e3d8cce0D3570E21FEf820F9Fee077777

@kronosapiens
Copy link
Member Author

kronosapiens commented Dec 16, 2024

@area do the docs need to be updated to go from Ethereum mainnet to Arbitrum mainnet?

Also, do we need to migrate off of InfuraProvider as IIRC that only support L1 chains?

https://docs.metamask.io/services/reference/arbitrum/quickstart/#ethers

@area
Copy link
Member

area commented Dec 18, 2024

@area do the docs need to be updated to go from Ethereum mainnet to Arbitrum mainnet?

I don't see why we shouldn't, I'm always going to be pro-keeping-docs-up-to-date.

Also, do we need to migrate off of InfuraProvider as IIRC that only support L1 chains?

That doesn't seem to be the case?. I'd be more than happy to regardless though as you can just pass an Infura endpoint to a 'normal' JSONRPCprovider, right?

@kronosapiens kronosapiens force-pushed the maint/goerli branch 2 times, most recently from a0bfc38 to 2fb750c Compare December 18, 2024 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove support for Goerli
2 participants