Skip to content

fix: don't broadcast variables from embedded apps#7896

Merged
akshayka merged 1 commit intomainfrom
aka/fix-embedded-app-notify
Jan 22, 2026
Merged

fix: don't broadcast variables from embedded apps#7896
akshayka merged 1 commit intomainfrom
aka/fix-embedded-app-notify

Conversation

@akshayka
Copy link
Contributor

This fixes a bug in which the variables panel was overridden by embedded apps' variables.

Frontends shouldn't know about embedded apps' variables and datasources.
@akshayka akshayka requested a review from dmadisetti as a code owner January 19, 2026 16:58
@vercel
Copy link

vercel bot commented Jan 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
marimo-docs Ready Ready Preview, Comment Jan 19, 2026 4:59pm

Request Review

@akshayka akshayka added the bug Something isn't working label Jan 19, 2026
@akshayka akshayka requested review from mscolnick and removed request for dmadisetti January 19, 2026 17:07
@akshayka akshayka merged commit 794b05f into main Jan 22, 2026
46 of 51 checks passed
@akshayka akshayka deleted the aka/fix-embedded-app-notify branch January 22, 2026 00:06
@github-actions
Copy link

🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.19.5-dev37

botterYosuke pushed a commit to botterYosuke/marimo that referenced this pull request Jan 22, 2026
This fixes a bug in which the variables panel was overridden by embedded
apps' variables.
botterYosuke pushed a commit to botterYosuke/marimo that referenced this pull request Jan 23, 2026
This fixes a bug in which the variables panel was overridden by embedded
apps' variables.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants