Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Explicit validators #287

Closed
meejah opened this issue Aug 31, 2020 · 0 comments · Fixed by #288
Closed

Explicit validators #287

meejah opened this issue Aug 31, 2020 · 0 comments · Fixed by #288

Comments

@meejah
Copy link
Collaborator

meejah commented Aug 31, 2020

Per #257 (comment) we end up using config setters to get the validation side-effect.

It also means we're doing an SQL INSERT and then soon after an UPDATE of the same information.

One way around this would be to have (separate, internal) validator methods that the setters AND the initializer calls. Perhaps there is some other way to achieve the same effect.

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

Successfully merging a pull request may close this issue.

1 participant