graphprotocol / graphprotocol/graph-node

[Bug] Disallow sorting by child fields that are arrays

Aperta
#5,207 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

bug Stale
Lingua principale
Rust
Stelle
3.2k
Fork
1.1k
Merge medio
4g 1h
PR unite (30g)
1

Descrizione

Bug report

In a subgraph schema like

type Parent {
  id: ID!
  children: [Child!]!
}

we allow a query like parents(orderBy: children__id) { .. } but that ordering doesn't make much sense. When a parent has multiple children, it's not clear which child's id we will use for the ordering. We should simply not support that.

Some information to help us out
  • Tick this box if this bug is caused by a regression found in the latest release.
  • Tick this box if this bug is specific to the hosted service.
  • I have searched the issue tracker to make sure this issue is not a duplicate.
OS information

None

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Nell'issue non sono indicati file, test o punto di ingresso. Inizia tracciando come vengono convalidati i campi GraphQL orderBy per le relazioni annidate, quindi riproduci il caso di schema Parent/children; il lavoro è completato quando l'ordinamento per un campo di tipo array di un elemento figlio viene rifiutato, mentre le relazioni scalari supportate rimangono valide.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
graphql, rust
Ambito
api
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.