-
Notifications
You must be signed in to change notification settings - Fork 19
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
Change Extra.forbid to Extra.ignore #9
Comments
The If extra fields are needed, one may try to extend the class for customizations. |
Hi, Extra.ignore will also disallow extra values, but it will not raise exceptions. If you want this library to be easy to interact with, I would suggest switching. I already wrapped this library in a lot if code to handle this, but it just adds a lot of extra complexity. |
This enables setting the extras option on a global scale. This enables the usecase of kuimono#14 while still being in the spirit of kuimono#9.
Hi there,
The usage of Extra.forbid is difficult to work with in other libraries, because it requires sanitizing data before interacting with this library. Is it possible to change Extra.forbid to Extra.ignore? Especially for Schema but also other models.
The text was updated successfully, but these errors were encountered: