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

Implement exceptions.PreventUpdate #25

Closed
cpsievert opened this issue Jan 19, 2018 · 3 comments · Fixed by #111
Closed

Implement exceptions.PreventUpdate #25

cpsievert opened this issue Jan 19, 2018 · 3 comments · Fixed by #111
Assignees
Milestone

Comments

@cpsievert
Copy link
Contributor

cpsievert commented Jan 19, 2018

plotly/dash#190

plotly/dash@fd2f353
plotly/dash@ad92fa3

@cpsievert cpsievert added this to the dasher release milestone Jan 19, 2018
@rpkyle
Copy link
Contributor

rpkyle commented Jul 15, 2019

@nicolaskruchten @alexcjohnson I was talking with @KPhans not that long ago, and he asked whether this was on the list of features to support. We haven't discussed this item specifically.

It exists in Dash for Python, so I am assuming that it is?

@alexcjohnson
Copy link
Collaborator

Definitely should be on the list. Also see dash.no_update plotly/dash#680 which is only strictly necessary when we get to multi outputs but is a nice alternative to raising an error even for single outputs.

@rpkyle rpkyle self-assigned this Aug 1, 2019
@rpkyle
Copy link
Contributor

rpkyle commented Aug 1, 2019

Definitely should be on the list. Also see dash.no_update plotly/dash#680 which is only strictly necessary when we get to multi outputs but is a nice alternative to raising an error even for single outputs.

We discussed the implementation of PreventUpdate in R, and I believe the initial plan is to use something like no_update. I'm working on that today, will attempt to draft a PR by tomorrow.

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

Successfully merging a pull request may close this issue.

3 participants