graphql-python/graphene-django

GraphQLView should detect batch requests

Open

#967 geöffnet am 21. Mai 2020

Auf GitHub ansehen
 (7 Kommentare) (1 Reaktion) (0 zugewiesene Personen)Python (4.173 Stars) (754 Forks)batch import
help wanted✨enhancement

Beschreibung

Is there a valid reason why the GraphQLView input is validated to either only ever accept batched requests or never accept batched requests. Is it not reasonable to accept both?

https://github.com/graphql-python/graphene-django/blob/ee120c48e16dda818cb253fdc36b7053402956b0/graphene_django/views.py#L217

Contributor Guide

Tech Stack
pythondjangographql
Domain
backendapi
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Basic Django knowledgeGraphQL basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80
Research-Richtung
The issue requests modifying the GraphQLView in graphene django/views.py to detect and accept both batched and single GraphQL requests. The relevant code is at line 217 (linked in the issue). Currently, the view validates the input to expect either a list or a single object, but not both. A first time contributor should examine the parse body or dispatch method to implement logic that checks the request body type (list or dict) and processes accordingly. The goal is to allow a client to send either format without breaking existing functionality.