Skip to content

Validate that MetaMixin can only be used with sync controller and the opposite #141

New issue
New issue
Closed
#161
Closed
Validate that MetaMixin can only be used with sync controller and the opposite#141
#161
Assignees
AlexanderZharyuk
Labels
enhancementNew feature or requestgood first issueGood for newcomers
@sobolevn

Description

@sobolevn
sobolevn
opened on Oct 22, 2025

We have a TODO task:

django-modern-rest/django_modern_rest/validation.py

Lines 233 to 237 in 489038f

def __call__(self, controller: 'type[Controller[BaseSerializer]]') -> bool:
"""Run the validation."""
# TODO: validate that sync controller have `MetaMixin`
# and async ones have `AsyncMetaMixin`
self._validate_components(controller)

It should not be possible to create

class My(MetaMixin, Controller[PydanticSerializer]):
    async def get(self) -> int: ...

and

class My(AsyncMetaMixin, Controller[PydanticSerializer]):
    def get(self) -> int: ...

Please, add a validation for this.

Metadata

Metadata

Assignees

Labels

enhancementNew feature or requestgood first issueGood for newcomers

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions