reown-com / appkit

The full stack toolkit to build onchain app UX
https://reown.com/appkit
Apache License 2.0
4.95k stars 1.41k forks source link

[bug] Ignored wagmi adapter connectors configuration #2978

Open yurymarozau opened 1 month ago

yurymarozau commented 1 month ago

Link to minimal reproducible example

https://codesandbox.io/p/devbox/jolly-sky-28xqlr?file=%2Fapp%2Fconfig%2Findex.tsx&workspaceId=24106f18-90d3-4483-abb3-4a747ebd3a29

Summary

Hello! After migration to reown all configuration according to the connectors in the WagmiAdapter config are being ignored. Example: Only injected connecters are configured, but all possible connection are used:

image

List of related npm package versions

"dependencies": { "@reown/appkit": "^1.0.5", "@reown/appkit-adapter-wagmi": "^1.0.5", "@tanstack/react-query": "^5.56.2", "next": "14.2.1", "react": "^18", "react-dom": "^18", "viem": "^2.21.15", "wagmi": "^2.12.14" }, "devDependencies": { "@types/node": "^20", "@types/react": "^18", "@types/react-dom": "^18", "autoprefixer": "^10.4.19", "eslint": "^8", "eslint-config-next": "14.2.1", "postcss": "^8", "tailwindcss": "^3.4.3", "typescript": "^5" }

yurymarozau commented 1 month ago

After https://github.com/reown-com/appkit/pull/2985 there are duplicated connectors in the config, but none of them are showing up.

image image image

Overall, issue still exists since I'm not able to control connectors (eg. if only WalletConnect connector is in the connectors list, there are injected connectors in the modal).

x0good commented 10 hours ago

I have the same issue (appkit 1.4.1 version). Set Wagmi according https://docs.reown.com/appkit/next/core/installation#wagmi-config

//Set up the Wagmi Adapter (Config)
export const wagmiAdapter = new WagmiAdapter({
  storage: createStorage({
    storage: cookieStorage
  }),
  ssr: true,
  projectId,
  networks
})

but ssr still false and cookie storage is not used

@enesozturk I see your merged MR https://github.com/reown-com/appkit/issues/2940 is it related somehow?