Beschrijf de fout
Bij meer dan 100 discussies worden diegenen boven de 100 niet getoond. Dit komt door het type query wat gedaan wordt. Dit heeft als voordeel dat per compile ronde maar 1x de GitHub GraphQL API aangesproken wordt.
Reproduceren
Stappen om de fout te reproduceren:
Ga naar een willekeurig document
Klik op voeg discussie toe
Voeg een bericht toe
Zie hoe het bericht niet verschijnt in de frontend
Verwacht gedrag
100 discussies zouden moeten kunnen
Extra context
Oorzaak is de query waarin we alle discussies 1x ophalen tijdens het compilen. Er kan per document een query gedaan worden maar daarmee wordt de load op GitHub groter. Hiermee vergroten we het aantal discussies naar 100 per document. Dit kan met de volgende query waarin bij category de documenttitel meegegeven wordt:
{
search(
first: 100
type:DISCUSSION,
query:"repo:fdnd/docs.fdnd.nl category:organisatie"){
discussions: edges {
discussion: node {
...on Discussion {
title
author {
login
}
comments {
totalCount
}
body
bodyHTML
url
}
}
}
}
}
Beschrijf de fout Bij meer dan 100 discussies worden diegenen boven de 100 niet getoond. Dit komt door het type query wat gedaan wordt. Dit heeft als voordeel dat per compile ronde maar 1x de GitHub GraphQL API aangesproken wordt.
Reproduceren Stappen om de fout te reproduceren:
Verwacht gedrag
Extra context Oorzaak is de query waarin we alle discussies 1x ophalen tijdens het compilen. Er kan per document een query gedaan worden maar daarmee wordt de load op GitHub groter. Hiermee vergroten we het aantal discussies naar 100 per document. Dit kan met de volgende query waarin bij category de documenttitel meegegeven wordt: { search( first: 100 type:DISCUSSION,
query:"repo:fdnd/docs.fdnd.nl category:organisatie"){ discussions: edges { discussion: node { ...on Discussion { title author { login } comments { totalCount } body bodyHTML url } } } } }