update starknet dependency version to 5.9.1 about starknet-react HOT 3 CLOSED

I agree with this. From our discussion on telegram my understanding is the following:

• get-starknet and wallets return an object that implements the AccountInterface v4
• starknet.js requires now the AccountInterface v5

My idea is that for the time being we should wrap the account v4 into a shim that implements the account v5 interface (and throws if the library/dev tries to use a new method).

What do you think?

from starknet-react.

I looked more into this tonight.

• Most changes are caused by library.chainId becoming await library.getChainId(). We can wrap this into a hook so the change is simple.
• get-starknet-core has starknet as a peer dependency. This is what seems to be causing issues.

At the moment and for the next few weeks, all injected wallets will still implement the account interface from v4.
So the only way I see adding support for v5 is by:

1. explicitly request a v4 account interface to the wallet
2. wrap the v4 account into a shim that implements the v5 interface
3. use the shim as the starknet-react provider

in a few weeks/months once v4 is officially deprecated, we can start requesting v5 and discard the shim.

What do you think?

from starknet-react.

v5 is now supported.

from starknet-react.