Skip to content

more robust error message for callback return validation #622

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 4, 2019
Merged

more robust error message for callback return validation #622

merged 1 commit into from
Mar 4, 2019

Conversation

alexcjohnson
Copy link
Collaborator

Fixes #621
Uses the __repr__ @T4rk1n introduced in #605 to more simply construct the error message:

InvalidCallbackReturnValue: 
The callback for `[<Output `c.children`>, <Output `d.children`>]`
returned a value having type `instance`
which is not JSON serializable.
...

@alexcjohnson alexcjohnson requested a review from T4rk1n March 2, 2019 01:13
@alexcjohnson alexcjohnson merged commit 708775d into master Mar 4, 2019
@alexcjohnson alexcjohnson deleted the multi-return-validation branch March 4, 2019 13:13
HammadTheOne pushed a commit to HammadTheOne/dash that referenced this pull request May 28, 2021
@Marc-Andre-Rivet
remove publish scripts & update contributing (plotly#622)
57c3a76
HammadTheOne pushed a commit that referenced this pull request Jul 23, 2021
@Marc-Andre-Rivet
remove publish scripts & update contributing (#622)
f8bb9cb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Marc-Andre-Rivet Marc-Andre-Rivet Marc-Andre-Rivet approved these changes

@T4rk1n T4rk1n Awaiting requested review from T4rk1n T4rk1n is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@alexcjohnson @Marc-Andre-Rivet