Github GraphQL #178
Draft
Github GraphQL #178
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
DavidBruant
commented
Jan 18, 2024
| }` | ||
|
|
||
| return this.graphQLCall(query) | ||
| .then(resp => resp.data.repository.deployments.nodes[0].statuses.nodes[0].state.toLowerString()) |
Contributor
Author
There was a problem hiding this comment.
on a troqué un gloubiboulga de .then et response.json()
contre cette ligne un peu affreuse (et la requête ci-dessus qui est plutôt cool)
DavidBruant
commented
Jan 18, 2024
| .then(response => response.json()) | ||
| .then(json => { | ||
| const statusesUrl = json[0].statuses_url | ||
| const [owner, name] = repoId.split('/') |
Contributor
Author
There was a problem hiding this comment.
l'histoire du repoId marchait bien avec les URLs REST, mais moins bien avec les requêtes GraphQL
Voir #174 pour une solution
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Dans un autre contexte, j'ai besoin de faire du GraphQL
Et vu que je connais pas, je me suis dis que j'allais apprendre
Et j'ai découvert que Github avait une API GraphQL, donc, je me suis dis que j'allais voir ce que ça donnait sur Scribouilli
Le premier commit a été un peu laborieux parce que je découvrais, mais en vrai, ça a l'air facile
J'imagine que tout traduire en GraphQL serait assez facile, mais je ne sais pas si ça a trop de valeur vu que nos appels sont assez simples
À l'exception peut-être de
getPagesWebsiteDeploymentStatusqui fait 2 appels successifs et j'imagine que ça serait moins laborieux en une requête GraphQL unique (et j'imagine que ça résoudrai le problème du cache directement aussi vu qu'il n'y a pas de cache HTTP en GraphQL)Un autre avantage, c'est qu'en GraphQL, on contrôle ce que l'API retourne et donc on peut être assez minimalistes (là où l'API REST balance des floppées d'information dont on n'a pas besoin par défaut)
Lors du premier commit, les appels REST et GraphQL cohabitent et ça n'a pas l'air d'être un problème du tout