-
-
Notifications
You must be signed in to change notification settings - Fork 40
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
feat: 209 global audio #241
Conversation
✅ Deploy Preview for cientos-tresjs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @JaimeTorrealba , the demo didn't work for me, so I couldn't try it out. I marked the line that the error sent me to and made some other remarks.
Once that's fixed, rerequest a review and I'll have another look. 🙂
Great. Everything works now and the other changes look good. 👍 I approved the review, but made one more comment above. I don't know if it's doable or within the scope of the work here, so feel free to ignore it. |
…se state when is playing
closes #209