Skip to content

Implémentation de la liaison et dissociation d'un ORCID

Qu'est-ce que le MR fait ?

Implémente les étapes de liaison ("addition") et dissociation ("removal") du "claim" d'ORCID.

Comment tester ?

Voici un document élaboré par @mpigeon avec plusieurs scénarios de tests du Claim : https://docs.google.com/document/d/1aHiYA0qw_BZU8apIIzX1IMhS7ii9YVLjYKmzkD19gks/edit?usp=sharing. J'y ai ajouté des liens vers des articles pour pouvoir tester. Ignorer les tests touchants à 'authentification pour le moment (ils seront traités dans la dernière MR).

Capture d'écran / vidéo

Git

  • Est-ce que le message de commit est clair et décrit bien les changements ?
  • Est-ce que le MR est exempt de commits qui modifient des commits qui ont été ajoutés dans le même MR ?

Correction de bogues

  • Est-ce qu'il y a un test de régression ?

Nouvelle fonctionnalité

  • Est-ce que la fonctionnalité est documentée et testée ?

Autres vérifications

  • Internationalisation
    • Est-ce que les chaînes de caractères sont internationalisées ?
    • Est-ce que la langue de base est le français ?
  • Changements au code
    • Est-ce que tous les print / console.log ont été enlevés ?
    • Est-ce que les requêtes HTTP ont un timeout explicite ?
  • Front-end
    • Est-ce que les classes adhèrent à la convention de nommage : traits d'union pour le Sass et camelCase pour JavaScript ?
    • Est-ce que le code suit notre guide javascript ?
  • URLs
    • Est-ce que les URLs contiennent uniquement des lettres et des caractères de soulignement ?
  • Templates
    • Est-ce que les noms des templates contiennent uniquement des lettres et des caractères de soulignement ?
    • Est-ce que les "partials" sont dans les répertoires "partials" ?

Related to #3030

Merge request reports