search

chiefwigms / picobrew_pico

MIT License
149 stars 63 forks source link

support unknown fields in the request body for ispindel #253

Closed tmack8001 closed 3 years ago

tmack8001 commented 3 years ago

resolves https://github.com/chiefwigms/picobrew_pico/issues/247

ref: https://webargs.readthedocs.io/en/latest/advanced.html#setting-unknown

chiefwigms commented 3 years ago

Explicitly include marshmallow in requirements.txt?

tmack8001 commented 3 years ago

Don't think we need to explicitly include marshmallow cause marshmallow is used under the hood for building out webargs which we already have an explicit dependency on. Wouldn't marshmallow come as a transitive dependency of webargs otherwise we would need to make sure the versions stay consistent whenever updating one or the other?