umutbozkurt / django-rest-framework-mongoengine

Mongoengine support for Django Rest Framework
MIT License
616 stars 167 forks source link

`style` not included in `utils.get_field_kwargs()`; not aligned to DRF core arguments #288

Open regoawt opened 2 years ago

regoawt commented 2 years ago

Currently, if style is defined as a kwarg on a mongoengine field, utils.get_field_kwargs() method does not build the DRF serializer field with it. To achieve this one has to wrap serializers.build_field() and update the field_kwargs dictionary, which seems unnecessary?

Is this intended behaviour?

The style argument is listed as a DRF 'core argument' on the base serializer field: https://www.django-rest-framework.org/api-guide/fields/#core-arguments

IATF commented 2 years ago

这是来自QQ邮箱的假期自动回复邮件。   您好,我最近正在休假中,无法亲自回复您的邮件。我将在假期结束后,尽快给您回复。

BurkovBA commented 2 years ago

@regoawt Hello.

I might be wrong, but I feel like style is a relatively new field in DRF.

Thank you for your PR .Please, add unit-tests to your changes, so that we could ensure their correctness.