Bug: Version 4.0.4 throws window not defined error on server rendered apps like next.js - Githubissues

coinbase / coinbase-wallet-sdk

An open protocol that lets users connect their mobile wallets to your DApp

https://coinbase.github.io/coinbase-wallet-sdk/

MIT License

1.4k stars 521 forks source link

Bug: Version 4.0.4 throws window not defined error on server rendered apps like next.js #1334

Closed cb-jake closed 1 week ago

cb-jake commented 1 week ago

Describe the bug

Version 4.0.4 throws window not defined error on server rendered apps like next.js

Steps

Install version 4.0.4 into a next.js app
Setup SDK
Noticed window not defined error.

Regression was most likely introduced in one of these PR's:

https://github.com/coinbase/coinbase-wallet-sdk/pull/1321 https://github.com/coinbase/coinbase-wallet-sdk/pull/1322

Expected behavior

SDK to not throw error. It should check for window to be defined

Version

4.0.4

Additional info

image (1)

Desktop

Chrome

Smartphone

No response