add Bitget wallet is discovery

starknet-io / get-starknet

StarkNet wallet <-> dApp bridge

MIT License

154 stars 106 forks source link

add Bitget wallet is discovery #221

Closed shuise closed 3 weeks ago

shuise commented 3 months ago

dan-ziv commented 2 months ago

Hey @shuise, Please fix failed test.

DetonateYin2077 commented 2 months ago

Hi @shuise , I noticed that you've added the Bitget Wallet configuration to the discovery.ts file. Much appreciate your work on this.

However, the test cases have not been updated to reflect this change. The length of discoveryWallets should now be 3, since the Bitget Wallet has been added.

To fix the failing test, the expected value should be updated from 2 to 3.

Please let me know if you have any other questions.