sourcegraph/sourcegraph-public-snapshot

insights: dashboards PageInfo incorrectly reports there are additional pages

Ouverte

#34 992 ouverte le 5 mai 2022

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)Go (1 374 forks)auto 404
backendbugcode-insightsgood first issuereliability

Métriques du dépôt

Stars
 (10 290 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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

Guide contributeur

Direction de recherche
Inspectez l'approche fonctionnelle dans InsightViewQueryConnectionResolver (lien dans le ticket) et appliquez une logique de pagination similaire a InsightsDashboardConnection pour corriger le bug PageInfo.
Stack technique
go
Domaine
backendapi
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
GoGraphQL
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75