Skip to content
This repository was archived by the owner on Jun 19, 2025. It is now read-only.

fix: use Endpoint ID instead of Chain ID#37

Open
0xemrekaya wants to merge 2 commits into
LayerZero-Labs:mainfrom
0xemrekaya:patch-1
Open

fix: use Endpoint ID instead of Chain ID#37
0xemrekaya wants to merge 2 commits into
LayerZero-Labs:mainfrom
0xemrekaya:patch-1

Conversation

@0xemrekaya

@0xemrekaya 0xemrekaya commented Mar 13, 2024

Copy link
Copy Markdown

Hi,

{
  "ethereum":       101,
  "arbitrum":       110,
  "optimism":       111,
  "goerli-mainnet": 154,
  "sepolia-mainnet": 161
}

is written as Chain IDs but these are endpoint ids.
Writing as chain id can confuse me and other developers.

  • chainIds.json to endpointIds.json
  • CHAIN_IDS to ENDPOINT_IDS

I just replaced them.

@0xemrekaya

Copy link
Copy Markdown
Author

cc: @sirarthurmoney and @TRileySchwarz

@0xemrekaya 0xemrekaya changed the title Use Endpoint ID instead of Chain ID fix: use Endpoint ID instead of Chain ID Mar 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant