strawberry-graphql/strawberry-django

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

Aperta

#544 aperta il 4 giu 2024

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)Python (152 fork)auto 404
enhancementhelp wanted

Metriche repository

Star
 (496 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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.

Guida contributor

Direzione di ricerca
Investiga la logica del filtro in strawberry django/filters.py quando handle django errors è abilitato, concentrandoti su come il tipo restituito cambia da lista a unione con OperationInfo e come ciò influisce sull'applicazione dei filtri.
Tech stack
pythondjango
Dominio
backendapidatabase
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonDjangoGraphQL
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70