strawberry-graphql/strawberry-django

`mutations.delete` and `mutations.update` do not support both `filters` _and_ `handle_django_errors`

Aberta

#544 aberto em 4 de jun. de 2024

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (0 responsável)Python (152 forks)auto 404
enhancementhelp wanted

Métricas do repositório

Stars
 (496 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Describe the Bug

When handle_django_errors is turned on (either globally or for an individual mutation) for mutations.delete and mutations.update then filters on those mutations has no effect.

System Information

  • Operating system: Ubuntu
  • Strawberry version: 0.234.0
  • Strawberry Django version: 0.42.0

Additional Context

The problem is here: https://github.com/strawberry-graphql/strawberry-django/blob/90e1878959d83af471fc20d8f4c611ce2e767fa8/strawberry_django/filters.py#L320-L321

When handle_django_errors is turned on, is_list is no longer true, because the return type is now a union with OperationInfo. Additionally, the return type is now no longer even a list.

Guia do colaborador

Direção de pesquisa
Investigue a lógica de filtro em strawberry django/filters.py quando handle django errors está habilitado, focando em como o tipo de retorno muda de lista para uma união com OperationInfo e como isso afeta a aplicação dos filtros.
Pilha de tecnologia
pythondjango
Domain
backendapidatabase
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PythonDjangoGraphQL
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70