Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[bug] Ignored wagmi adapter connectors configuration #2978

Open
yurymarozau opened this issue Oct 1, 2024 · 4 comments
Open

[bug] Ignored wagmi adapter connectors configuration #2978

yurymarozau opened this issue Oct 1, 2024 · 4 comments
Labels
bug Something isn't working needs review

Comments

@yurymarozau
Copy link

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 yurymarozau added bug Something isn't working needs review labels Oct 1, 2024
@yurymarozau
Copy link
Author

After #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
Copy link

x0good commented Nov 26, 2024

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 #2940
is it related somehow?

@rtomas
Copy link
Contributor

rtomas commented Dec 11, 2024

@x0good can you confirm that is working in latest ? 1.5.3 ?

@x0good
Copy link

x0good commented Dec 11, 2024

@x0good can you confirm that is working in latest ? 1.5.3 ?

I fixed my usecase and already created MR here reown-com/reown-docs#177

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs review
Projects
None yet
Development

No branches or pull requests

3 participants