sourcegraph/sourcegraph-public-snapshot

insights: dashboards PageInfo incorrectly reports there are additional pages

Aperta

#34.992 aperta il 5 mag 2022

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)Go (1374 fork)auto 404
backendbugcode-insightsgood first issuereliability

Metriche repository

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

Descrizione

When any of the arguments first, after or ID are passed to InsightsDashboardConnection the pageInfo returned will always indicate that there is an additional page. This is not always correct.

see InsightViewQueryConnectionResolver for working approach

Guida contributor

Direzione di ricerca
Ispeziona l'approccio funzionante in InsightViewQueryConnectionResolver (link nell'issue) e applica una logica di paginazione simile a InsightsDashboardConnection per correggere il bug di PageInfo.
Tech stack
go
Dominio
backendapi
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.
Abbastanza chiara
Prerequisiti
GoGraphQL
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75