-
-
Notifications
You must be signed in to change notification settings - Fork 2k
Open
Labels
documentationDocumentation related issue.Documentation related issue.
Description
Suggestion
While working with ethers v6, given chainId 42 it's returning Kovan.
Kovan is now deprecated, and LUKSO Mainnet uses chainId 42. More info here: https://docs.lukso.tech/networks/mainnet/parameters
Keeping it this way will return inaccurate information to developers/users. (Example in a dApp ⬇️ )
ethers.js/src.ts/providers/network.ts
Line 389 in 6017d3d
registerEth("kovan", 42, { ensNetwork: 42 }); |
I am happy to do the PR, if the change is approved.
Metadata
Metadata
Assignees
Labels
documentationDocumentation related issue.Documentation related issue.