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

Await send_custom_message in disable #9

Closed
wants to merge 2 commits into from
Closed

Conversation

cs42
Copy link

@cs42 cs42 commented Jan 8, 2025

Should solve bug #8, implements await similar to the version in enable()

@gadenbuie
Copy link
Contributor

Thanks @cs42, I appreciate that you took a look at this and submitted a PR! Unfortunately, the fix is both simpler and more complicated. Ultimately, the issue arises because shiny's session.send_custom_message() method is asynchronous, but we don't really want to force app authors to await .disable(). (The other issue with this PR is that we don't want .disable() to create a new reactive effect since the goal is to clean up previously created effects.)

I ended up implementing a fix in a separate PR #10. Thanks again!

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

Successfully merging this pull request may close these issues.

2 participants