Skip to content

Update dependencies #23

New issue

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

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

Already on GitHub? # to your account

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Conversation

on3iro
Copy link

@on3iro on3iro commented Jun 25, 2023

Hey @itaditya ,

I tried to updated most dependencies and also updated the Root component type. This should fix the issues mentioned in #21 and #22 . However I didn't have much time and wasn't able to figure out why the audio checkbox in the demo page can't be clicked (which also makes a test fail).

It would be cool if you could give me a few pointers on where to look are if you would just take over from here :)
I haven't squashed my changes, yet, as I consider this to be a draft for now.

let me know, what you think.

<Suspense fallback={<Loader size="large" />}>
<Suspense
fallback={<Loader size="large" />}
children={undefined}
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should probably be fixed - my IDE showed me an error here, but shouldn't the div below technically be considered to be children? 🤔

@itaditya
Copy link
Owner

Thanks @on3iro, I'll check it out

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants