Connector onNetworkChanged function doesn't support text chainIds

[dhruv035]

The InjectedConnector onNetworkChanged handler throws an error if I have my wallet network chainId set as 'SN_SEPOLIA' instead of a bigint, it would make sense to parse said string on the connector if configured like so on the wallets

[fracek]

It used to do this, but AFAIK the new wallet spec specifies that the chainId argument should be an hex-encoded chain id.

https://github.com/starknet-io/types-js/blob/229373398097b9bd85c3d7a42443d1b7310f258b/src/wallet-api/events.ts#L4-L10

[ShantelPeters]

Please can I work on this @fracek

[fracek]

Hey @dhruv035 what wallet is emitting the old chain id? So that I can test it.

[dhruv035]

Hey sorry I missed this comment, I was using Argent wallet, it may have been an older version admittedly but I think backwards support for it would make sense still. Sorry for the late response kinda lost this comment in mail among the so many others on the other issue