Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ajoute la mention du scope minimal du token GitHub #6185

Merged
merged 3 commits into from
Jan 22, 2022

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Oct 3, 2021

Le dépôt de ZdS étant public, le token doit au moins avoir le scope public_repo (cf. https://docs.github.com/en/developers/apps/building-oauth-apps/scopes-for-oauth-apps#available-scopes). Le token a tous les droits du compte GitHub, dont la création d'issues, mais limité aux dépôts publics.

Fix #6100.

Contrôle qualité

Tester l'affichage dans le profil :

  • Se connecter avec admin
  • Aller sur l'administration Django
  • Faire en sorte qu'admin soit dans le groupe devs
  • Constater que dans le profil, la page "token github" est bien à jour avec le nouveau texte.

Tester que c'est le bon scope :

  • Créer un dépôt public bidon sur son compte github
  • Générer un token pour votre instance locale avec le scope public_scope
  • Enregistrer le token pour l'utilisateur "admin"
  • Créer une issue depuis l'instance locale
  • Constater que l'issue est bien créée sur GitHub

@Arnaud-D Arnaud-D added the hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest label Oct 3, 2021
@coveralls
Copy link

coveralls commented Oct 3, 2021

Coverage Status

Coverage remained the same at 87.019% when pulling 82a600a on Arnaud-D:going_public into 18a1991 on zestedesavoir:dev.

templates/member/settings/github.html Outdated Show resolved Hide resolved
Co-authored-by: Situphen <Situphen@users.noreply.github.com>
@Arnaud-D Arnaud-D requested review from AmauryCarrade and removed request for Situphen November 8, 2021 20:28
Copy link
Member

@Situphen Situphen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA OK Ça fonctionne correctement !

@Situphen Situphen merged commit 0cf61a9 into zestedesavoir:dev Jan 22, 2022
@Arnaud-D Arnaud-D deleted the going_public branch February 13, 2022 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Préciser quel doit être le scope du token GitHub
4 participants