-
Notifications
You must be signed in to change notification settings - Fork 18
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
Shiny Client Errors: Cannot read properties of null (reading 'deps') #128
Comments
similar issue: r-wasm/webr#330 |
Minimal reprex
|
I believe I have tracked down the underlying issue. That is, I now know how and why the problematic The issue is related to the patches made to webR's distribution of Shiny so that it can work well under WebAssembly. I have fixed the problem and made a new release of the package at https://github.com/r-wasm/shiny/releases. With this latest build, the example app works without generating any errors: Shinylive won't automatically pick up the change, but I will include upgrading the Shiny R package as part of some other work I am currently undertaking. Once that work has all been merged, it should fix the issue on shinylive.io etc. |
@georgestagg I think this fixes an issue I've been experiencing that I originally opened here ramnathv/htmlwidgets#487, but now lives here seanbirchall/reprexIDE#1. I created a bandaid solution using a delay that works ok... but I originally developed this mini htmlwidget so that I could avoid using some type of |
Yes, I think the raising of the The underlying issue was in my Wasm patches to R Shiny so that it can run in Shinylive. The patches did not correctly raise conditions to be handled, instead internally returning a It should now (hopefully) be fixed on https://shinylive.io/r/examples/, and once #131 is merged also in the outputs of the R Shinylive package and Quarto documents. I'll close the issue once that happens. @seanbirchall If the issue still remains after that point please feel free to open a new issue at posit-dev/shinylive. |
https://shinylive.io/r/editor/#code=NobwRAdghgtgpmAXGKAHVA6ASmANGAYwHsIAXOMpMAGwEsAjAJykYE8AKcqajGIgEwCu1OAGcMAcwpxm1AJQAdCHSYsOo1CwDWdCHEUQlaVAAIAPAFoTtCLVLslJk-yikoJgLwmu1APou3dgBJLCCAZU9rRlpROVxHEz4hEVFIpOExBwgnJ1IYXwA3FlooehFfJiIAd1EZBzAANWLSkRMAIUZq2sYFMAMnAwMlUQALG1YAQXR2YwASQVpcEznugpkDMABfAF0gA
I think it's due to this change in the
htmlwidgets
: r-wasm/htmlwidgets@892798fconsole log
When I enter the JS debugger I can see the following:
data
isnull
thusdata.deps
throwsOutside of shinylive everything is fine. Entering debugger in the same place does not show null value.
I'm trying to create a base shiny example for this to debug this easier but the example above shows that it is possible to have null value there so it would make sense to handle this somehow.
I have seen #124 but adding loading dependency from external CDS does not help.
The text was updated successfully, but these errors were encountered: