txnlab / use-wallet-js Goto Github PK

Getting this error:

Does Exodus require additional setup, beyond this?

// vue
app.use(WalletManagerPlugin, {
    wallets: [
        WalletId.EXODUS,
    ],
    network: NetworkId.MAINNET
})

In the core library and/or framework adapters, prevent initialization if window and localStorage are not available.

It also might make sense to include SSR framework examples, e.g. Next, Nuxt, etc

This issue lists Renovate updates and detected dependencies. Read the Dependency Dashboard docs to learn more.

Awaiting Schedule

These updates are awaiting their schedule. Click on a checkbox to get an update now.

• chore(deps): update dependency @testing-library/react to v15
• chore(deps): update dependency eslint to v9
• chore(deps): lock file maintenance

Open

These updates have all been created already. Click a checkbox below to force a retry/rebase of any.

• chore(deps): update all non-major dependencies (@testing-library/react, @types/node, @types/react, @walletconnect/sign-client, @walletconnect/types, @walletconnect/utils, lute-connect, typescript, vite, vitest, vue)
• chore(deps): update typescript-eslint monorepo to v7 (major) (@typescript-eslint/eslint-plugin, @typescript-eslint/parser)

Detected dependencies

• Check this box to trigger a request for Renovate to run again on this repository

this value for signTransactions and transactionSigner is manager instead of the corresponding wallet.

Leads to this error:

My workaround:

        const fn = manager.signTransactions.bind(manager.activeWallet)
        return fn(txnGroup, indexesToSign, returnGroup)

Affects the react package as well. If this is a commonly used pattern (returning a function from a getter), you may want to go over them and confirm the this value is as expected.

Following the vue example, I set the network to NetworkId.MAINNET

app.use(WalletManagerPlugin, {
  wallets: [
    WalletId.PERA,
  ],
  network: NetworkId.MAINNET
})

but if i log the network it's always testnet

const wallet = useWallet()
cl(wallet.activeNetwork) // 'testnet'

The networks in wallet.wallets[].activeNetwork are correctly 'mainnet'

EDIT:

Hook data (walletState, walletManager) aren't reactive.

1. For walletState, the issue is you can't reassign a reactive object

https://vuejs.org/guide/essentials/reactivity-fundamentals.html#limitations-of-reactive

2. For walletManager, it's a bit more tricky, but you have to wrap it with a ref and somehow trigger a reactive update.