0xMiden / 0xMiden/wallet-adapter
Feedback: Miden Wallet connection not working.
- Langage dominant
- TypeScript
- Étoiles
- 2
- Forks
- 9
- Merge moyen
- 25 min
- PR mergées (30 j)
- 1
Description
I tried to connect Miden Wallet to my dApp using instructions in the README but it didn't work as expected, my wallet adapter never leaves the `NotDetected` state, the same applies to this demo here: https://github.com/demox-labs/miden-wallet-demo
I rooted the issue to the browser extension injecting the `tridentWallet` while this package looks for `midenWallet` thus never catching up proper Miden Wallet availability: https://github.com/demox-labs/miden-wallet-adapter/blob/main/packages/wallets/miden/adapter.ts#L83
I was able to make it work using this trick:
```js
useEffect(() => {
window.midenWallet = window.tridentWallet;
}, []);
```
You should also mention in the docs that it is required to import the `reactui` package styles to make the `WalletMultiButton` working as expected:
```js
import "@demox-labs/miden-wallet-adapter-reactui/dist/styles.css";
```
Last detail, when right-clicking on the browser extension icon and selecting the Miden Wallet option, I get redirected to https://leo.app/ instead of https://miden.fi/
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Le problème se trouve dans le fichier adapter.ts à la ligne 83, où le code recherche `window.midenWallet` mais l'extension injecte `window.tridentWallet`. Commencez par examiner le fichier adapter.ts dans le package du portefeuille Miden. Vérifiez le fonctionnement de la détection du portefeuille et mettez à jour le nom de la propriété. Vérifiez également le README pour des instructions d'importation de styles manquantes et testez la correction avec le dépôt de démonstration fourni.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, react, typescript
- Domaine
- documentation, frontend
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 70/100