KnpLabs / KnpLabs/php-github-api
ResultPager fetchall on apps/installations incorrect result
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 2.2k
- Forks
- 585
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
See https://platform.github.community/t/pagination-differs-in-installations-api/1862
The result is an array with a count field and and subarray of all installations but we merge result when using the fetchAll method
We need to check if this is something we need/can fix in code or if we should add some docs about this issue
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die fetchAll-Behandlung von ResultPager für den Endpunkt apps/installations nachzuverfolgen, und vergleiche sie mit der in der Dokumentation beschriebenen Struktur aus Array, count und installations für diesen Endpunkt. Ermittle, ob der Client diese Struktur korrekt zurückgeben kann oder ob das Verhalten dokumentiert werden muss; abgeschlossen ist die Aufgabe, wenn diese Entscheidung getroffen und das daraus resultierende Verhalten oder die entsprechende Anleitung abgedeckt ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- github, php
- Bereich
- api
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 35/100