Skip to content
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

Add library loading capacity to _allow_dynamic_callbacks. #2789

Merged
merged 5 commits into from
Aug 27, 2024

Conversation

T4rk1n
Copy link
Contributor

@T4rk1n T4rk1n commented Mar 11, 2024

When _allow_dynamic_callbacks=True, callback can now import a component library and automatically loaded on the callback returns. Same restrictions as adding callbacks during callbacks, only guaranteed to work for a single user, meant to be used for development purposes.

@ndrezn
Copy link
Member

ndrezn commented Mar 12, 2024

@chriddyp / @hatched I think we should make a build of Dash based on this version and test whether we resolve the issue in App Studio with third-party component libraries. What do you guys think? Happy to create that build if so.

@gvwilson gvwilson self-assigned this Jul 25, 2024
@gvwilson gvwilson removed their assignment Aug 2, 2024
@gvwilson gvwilson added feature something new P2 considered for next cycle labels Aug 13, 2024
@ndrezn
Copy link
Member

ndrezn commented Aug 22, 2024

@T4rk1n: From @robertclaus, this change was tested in App Studio and works as expected 🥳 . See: https://github.com/plotly/notebook-to-app/issues/401#issuecomment-2269443734

@T4rk1n T4rk1n merged commit 870c661 into dev Aug 27, 2024
3 checks passed
@T4rk1n T4rk1n deleted the load-inside-callback branch August 27, 2024 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature something new P2 considered for next cycle
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants