-
Notifications
You must be signed in to change notification settings - Fork 35
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
Support SSR #113
Comments
fork with some work in this direction from @joeyiny |
Making progress! |
@joeyiny nice! feel free to open up a PR if you'd like to contribute your enhancements. Also FYI v2 dropped today. You'll def want to rebase your fork off of the latest main. Let me know if you have any questions about the migration, but it shouldn't be too too different. The README has been updated with the latest information |
@E-Kuerschner Oh nice, was actually planning on messaging you about v2! Definitely going to hop in first thing tomorrow hopefully. |
Currently working approach is to exclude it from SSR with window presence check (using useAudioPlayer@v2)
|
any example with |
Will likely be adding the 'use client' directive for general support |
It isn't clear how this hook will work with SSR frameworks like Next, Gatsby or Remix. Some users in the past have reported issues, likely due to the fact that this library has pretty explicit dependency on the global window object of the browser. It will help a lot of developers if this lib were to support these frameworks since the ecosystem is really shifting in that direction and away from large SPAs
The text was updated successfully, but these errors were encountered: