Jérôme Lafréchoux

Results 373 comments of Jérôme Lafréchoux

I guess this question is answered. Feel free to reopen otherwise.

I'm interested in your take on this. I'm not so happy with the implementation in flask-smorest. It does the job, but it's a bit awkward and perhaps a bit of...

Thank you for contributing. Closing this because I'd rather take another approach. See #252.

Hi. Yes, PR welcome. Thanks. The change described in https://github.com/marshmallow-code/flask-smorest/issues/252#issuecomment-1197287924 is fine to me. We could try to achieve this with a dict by doing the same with a namespace...

Hi. You may send a PR with field description. You shouldn't have permission issues creating a PR.

I'd say the error lies in container fields such as `List` field. ```py def _serialize(self, value, attr, obj, **kwargs) -> list[typing.Any] | None: if value is None: return None return...

> Being able to use the `load_default` logic when receiving a `None` value would probably already cover many usecases... I believe this is the point of https://github.com/marshmallow-code/marshmallow/pull/1381 (from https://github.com/marshmallow-code/marshmallow/issues/713#issuecomment-529154890).

> Any chance this will make it into a release soon-ish? :) I won't be working on it in the coming weeks. If anyone wants to take over: - rework...

I think this is ready to go. I removed the section about `ordered` from the quickstart because it is outdated and the option is now only required for a very...

TODO: - [x] Add test showing how to specify `OderedDict` as a dict class as class attribute. - [ ] Deprecate `ordered` - [ ] Deprecate `dict_class` argument of `get_declared_fields`