Partial param for Schema loading about flask_accepts

[volfpeter]

Hi @apryor6, @vladarefiev

Would it be possible to undo #81 ?

Here is the problem: the default value of partial in accepts() is False (the only correct default would be None, meaning let the schema do its job), and this argument is directly passed to schema.load(), overriding the schema instance's partial property and effectively breaking the passed in schema instance.

Keeping this argument with a default value of None is acceptable, but I would avoid it because it would still violate several best practices.

Br,
Peter

[volfpeter]

Also, see the suggested solution to this exact problem in #62

[apryor6]

Peter, That makes perfect sense. Happy to accept a PR that reconciles. I can get to it sooner or later, but it will be more challenging to find time than one might like for personal reasons.

…

On Aug 26, 2020, at 3:14 AM, Peter Volf ***@***.***> wrote:  Hi @apryor6, @vladarefiev Would it be possible to undo #81 ? Here is the problem: the default value of partial in accepts() is False (the only correct default would be None, meaning let the schema do its job), and this argument is directly passed to schema.load(), overriding the schema instance's partial property and effectively breaking the passed in schema instance. Keeping this argument with a default value of None is acceptable, but I would avoid it because it would still violate several best practices. Br, Peter — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

[volfpeter]

Hi,

I can submit a PR that removes the partial argument and restores the original state if that's okay for you.