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

Expose validate() #14

Closed
wolfgangwalther opened this issue Oct 5, 2022 · 3 comments · Fixed by #32
Closed

Expose validate() #14

wolfgangwalther opened this issue Oct 5, 2022 · 3 comments · Fixed by #32
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@wolfgangwalther
Copy link

Currently, only is_valid() is exposed, which just returns a bool.

It would be great to expose validate(), too, to get actual validation errors thrown (with information about why the validation is failing...).

@wolfgangwalther wolfgangwalther added the enhancement New feature or request label Oct 5, 2022
@olirice olirice added help wanted Extra attention is needed good first issue Good for newcomers and removed help wanted Extra attention is needed labels Nov 7, 2022
@duguorong009
Copy link
Contributor

@wolfgangwalther @olirice
I would like to contribute to the project, and take this issue.
Can you take a look at PR #32 ?

@zakandrewking
Copy link

zakandrewking commented Jul 13, 2023

I wonder if this would make it possible to return meaningful validation errors for CHECK constraints. For now, all we get for an invalid insert is:

ERROR:  new row for relation "node" violates check constraint "node_column_check"
DETAIL:  Failing row contains (... some data ...).

@olirice
Copy link
Collaborator

olirice commented Jul 20, 2023

resolved by #32

@olirice olirice closed this as completed Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants