Skip to content

Conversation

@LSCRSL
Copy link

@LSCRSL LSCRSL commented Apr 24, 2025

Ajout de la fonction pour revoker le token d'accès Github (uniquement), cette fonction a été ajouté dans github.js.
Création de la route vers cette fonction dans main.js.
Celle-ci pourra être appelée dans le code Scribouilli.

Ceci est censé fermer l'issue #17

@DavidBruant
Copy link
Contributor

Bonjour

Merci beaucoup pour la PR 🙏

Je réponds rapidement pour ne pas laisser la PR sans réponse

J'ai vu qu'il y a une PR et je ne vais pas encore prendre le temps de la regarder parce que je priorise d'autres choses
Je la regarderai sûrement la semaine du 5 mai

@DavidBruant
Copy link
Contributor

Bonjour,

Merci encore pour la PR et désolé du délai

Le code m'a l'air correct
Il y a des petites choses à améliorer, mais je pourrais en parler plus tard

J'aimerais tester le code. Je vois que dans ton nom de PR, il y a écrit "test passe". Est-ce que tu as rajouté un test automatisé ou est-ce qu'il s'agit de test sur ta machine ?

Dans tous les cas, pour tester la PR, je vais faire ce que j'avais écrit ici : #17 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants