ben-pr-p
ben-pr-p9mo ago

It looks like the client is still

It looks like the client is still attempted to connect to wss://mercedes.uaw.link/ws?session_id=87892f8b-ddf2-4613-b02b-c8a526b9300b, but the server can't read that because it's behind /path
7 Replies
ben-pr-p
ben-pr-pOP9mo ago
GitHub
marimo/frontend/src/core/websocket/useMarimoWebSocket.tsx at d12191...
A reactive notebook for Python — run reproducible experiments, execute as a script, deploy as an app, and version with git. - marimo-team/marimo
Myles Scolnick
Myles Scolnick9mo ago
yea for this case, we probably should move everything up to the base path, including the ws i can fix this i also didnt move /health - ill need to recall why i didnt move the whole app
ben-pr-p
ben-pr-pOP9mo ago
Thanks! Let me know if it's useful to make an issue or anything else
Myles Scolnick
Myles Scolnick9mo ago
@ben-pr-p i have the fix, but included it in https://github.com/marimo-team/marimo/pull/1002
GitHub
feat: create_asgi_app to programatically run marimo apps by mscolni...
This exposes a new function from the marimo module: create_asgi_app. Which is a builder to add many marimo apps: Closes #904 #395 Regarding the API: i chose a builder instead of seperate create_as...
Myles Scolnick
Myles Scolnick9mo ago
hopefully will be able to release this on wednesday when @Akshay is back from ooo
ben-pr-p
ben-pr-pOP9mo ago
Thank you so much! My users are auto workers, so these things like URL shortness and memorability actually make a huge difference I’m happy to report I’ve been able to build things with Marimo where they are totally comfortable with the UX though
Myles Scolnick
Myles Scolnick9mo ago
Yea and you can make the index page another marimo app that links out to each. Or just html/css