Hypertopic / Porphyry

Corpus analyses confrontation
https://hypertopic.org/porphyry
GNU Affero General Public License v3.0
21 stars 165 forks source link

FEATURE: Authorize a contributor to edit a viewpoint (see #183) #515

Closed paasshme closed 3 years ago

paasshme commented 3 years ago

Content

Implementation of issue #183, made with @Anas9820.

paasshme commented 3 years ago

@benel @Anas9820 J'ai fait les modifications que vous m'aviez demander: Ajout de la feature (sans fix) Puis ajout du premier fix (discussion ici: https://github.com/Hypertopic/Porphyry/pull/515#discussion_r645557125 )

Après avoir revérifier le comportement de la feature, il s'avère que le refactor que nous avions fait règle le second fix, le code n'était donc plus utile je l'ai donc supprimé. (discussion: https://github.com/Hypertopic/Porphyry/pull/515#discussion_r645555424 )

benel commented 3 years ago

Merci @JacquesMironneau. Comme ça, en deux commits, c'est très propre !

@JacquesMironneau @Anas9820 Je suis en train de tester votre code. Ça fonctionne très bien ! Bravo !

Par contre, quand je le récupère par Git, l'auteur est un certain "PashmiDev", c'est normal ? Par ailleurs, vous n'avez pas utilisé la syntaxe qui permet dans GitHub de déclarer un co-auteur dans un commit (Co-authored-by:), c'est normal ?

paasshme commented 3 years ago

@benel C'était l'ancien pseudonyme associé à mon compte github !

Par ailleurs, vous n'avez pas utilisé la syntaxe qui permet dans GitHub de déclarer un co-auteur dans un commit (Co-authored-by:), c'est normal ?

Je n'ai pas pensé à cette fonctionnalité, voulez-vous que je fasse la modification ?

benel commented 3 years ago

C'était l'ancien pseudonyme associé à mon compte github !

Si vous le souhaitez, vous pouvez changer la configuration de Git sur votre ordinateur (soit globalement, soit pour un dossier donné). À vous de voir...

Je n'ai pas pensé à cette fonctionnalité, voulez-vous que je fasse la modification ?

Ben c'est surtout pour votre collègue...

Pour l'instant, ça ne vous prendra que deux minutes à modifier (surtout si vous le faites à coup de cherry-pick et d'amend). Alors que plus ça ira et plus ça deviendra difficile à modifier (Git c'est un arbre de Merkle comme la Blockchain).

Anas9820 commented 3 years ago

Il est vrai que je mérite une certaine reconnaissance également :D

paasshme commented 3 years ago

@benel Après quelques erreurs avec le cherry-pick les modifications sont faites (auteur et co-auteur)

paasshme commented 3 years ago

Pour l'instant, ça ne vous prendra que deux minutes à modifier (surtout si vous le faites à coup de cherry-pick et d'amend)

J'ai pu tester également le git rebase -i HEAD~2, il permet de facilement faire une modification de message de commit (en utilisant les fonctions disponibles: dans notre cas le reword).

paasshme commented 3 years ago

@benel est-ce que ces changements vous conviennent ?

benel commented 3 years ago

@JacquesMironneau Les grands esprits se rencontrent ;) J'ai intégré votre contribution juste avant que vous ne posiez la question. :)

benel commented 3 years ago

Je fais les derniers tests avant de vous féliciter officiellement ;)