Skip to content

Conversation

@venkateshv1266
Copy link
Contributor

@venkateshv1266 venkateshv1266 commented Oct 29, 2025

TICKET: COIN-6224 COIN-6260

Update CELO Testnet configs

  • ChainId updated from 44787 to 11142220
  • Updated all transaction hex strings related to celo
  • Explorer url updated to https://sepolia.celoscan.io/

@venkateshv1266 venkateshv1266 force-pushed the COIN-6224-fix-celo-config branch from 40a8664 to 0c1c875 Compare October 29, 2025 08:07
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Celo Testnet configuration to migrate from the deprecated Alfajores testnet to the new Sepolia-based testnet. The primary changes include updating the chain ID from 44787 to 11142220 and updating the explorer URLs to point to the new Sepolia testnet infrastructure.

  • Updated CeloTestnet chain ID from 44787 to 11142220
  • Updated explorer URLs from Alfajores Blockscout to Sepolia Celoscan
  • Updated all test fixtures and expected values to reflect the new chain ID

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
modules/statics/src/networks.ts Updated CeloTestnet configuration with new chain ID and Sepolia explorer URLs
modules/statics/src/map.ts Updated chain ID mapping from 44787 to 11142220 for 'tcelo'
modules/sdk-coin-celo/test/unit/transactionBuilder/walletInitialization.ts Updated test assertions for new chain ID and transaction hashes; added debug console.log
modules/sdk-coin-celo/test/unit/transactionBuilder/send.ts Updated expected v value in test to match new chain ID
modules/sdk-coin-celo/test/unit/transactionBuilder/addressInitialization.ts Updated test assertion for new chain ID
modules/sdk-coin-celo/test/resources/celo.ts Updated all test fixture data (serialized transactions, broadcast formats) to reflect new chain ID

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@venkateshv1266 venkateshv1266 marked this pull request as ready for review October 29, 2025 08:40
@venkateshv1266 venkateshv1266 requested review from a team as code owners October 29, 2025 08:40
Copy link
Contributor

@nvrakesh06 nvrakesh06 left a comment

Choose a reason for hiding this comment

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

Looks good

Copy link
Contributor

@shivam-bitgo shivam-bitgo left a comment

Choose a reason for hiding this comment

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

lgtm

@venkateshv1266 venkateshv1266 merged commit dc12cab into master Oct 30, 2025
13 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.

5 participants