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

Les auteurs de tutos ne sont pas historisés #3078

Closed
SpaceFox opened this issue Oct 13, 2015 · 18 comments
Closed

Les auteurs de tutos ne sont pas historisés #3078

SpaceFox opened this issue Oct 13, 2015 · 18 comments
Assignees
Labels
C-Back Concerne le back-end Django S-BUG Corrige un problème
Milestone

Comments

@SpaceFox
Copy link
Contributor

Les auteurs de tutos sont attaché au tuto dans son ensemble.

Donc, si un auteur A a écrit une version 1 d'un tuto (publiée), et qu'il ajoute un auteur B pour préparer une version 2, ce nouvel auteur est crédité sur la version 1 y compris en sur la version publiée.

Un auteur ne devrait être crédité que sur les versions à partir du moment où il a été ajouté au tuto.

Exemple avec ce tuto que j'ai écrit seul mais auquel j'ai ajouté Javier pour une v2. Javier se retrouve en co-auteur de la v1.

@SpaceFox SpaceFox added S-BUG Corrige un problème C-Back Concerne le back-end Django labels Oct 13, 2015
@artragis
Copy link
Member

Bien reçu. Notons que grâce au modèle de la zep-12, le temps de correction de ce bug ne dépassera pas la demie heure et que le plus long risque d'être le temps d'écriture du TU.

@Eskimon
Copy link
Contributor

Eskimon commented Oct 13, 2015

Et ça c'est beau ! (petit larme)

@pierre-24
Copy link
Member

Moi, j'ai quand même l'impression que c'est dans le scope de la ZEP-08, non ? (parce que versionner les auteurs, c'est rajouter une ligne dans le manifest, toussa ... ?!?)

@Eskimon
Copy link
Contributor

Eskimon commented Oct 13, 2015

Nop, la zep-8 c'est plutôt "Comment githubiser nos repos" (accès externe, accès tiers etc...)

@GerardPaligot
Copy link
Member

Bien reçu. Notons que grâce au modèle de la zep-12, le temps de correction de ce bug ne dépassera pas la demie heure et que le plus long risque d'être le temps d'écriture du TU.

Versionner les auteurs ne prendrait que 30 minutes ? Ca semble quand même utopique (mais je peux me tromper).

@artragis
Copy link
Member

Non pierre ! ne te firminise pas ! Il ne s'agit pas de "versionner" les
auteurs mais de "figer leur état" lors de la publication. Je rappelle que
nous avons un objet appelé "PublishedContent" qui "fige l'état des choses
lors de la publication", il suffit donc d'ajouter les auteurs à "l'état des
choses" et tout roulera.

2015-10-13 18:57 GMT+02:00 Eskimon notifications@github.com:

Nop, la zep-8 c'est plutôt "Comment githubiser nos repos" (accès externe,
accès tiers etc...)


Reply to this email directly or view it on GitHub
#3078 (comment)
.

@pierre-24
Copy link
Member

aaaah, j'avais pas compris ça comme ça :p

@GerardPaligot
Copy link
Member

A mon avis, @SpaceFox ne parlait pas spécifiquement de la version publiée.

@SpaceFox
Copy link
Contributor Author

Oui et non. Afficher les vrais auteurs du moment sur la version publiée est le minimum, avoir les vrais auteurs partout est un bonus appréciable.

@artragis
Copy link
Member

pour revenir là dessus :

  • chaque commit (donc changement dans le code) est lié à son auteur dans l'historique des tutos
  • pour que l'auteur apparaisse dans la liste des auteurs, il faut qu'il soit (encore) inscrit sur le site

En fait l'idée de ne "figer les auteurs que pour la version publique" permet de dire au publique "dans al version que vous regardez, il y a eu ces auteurs là". Ce qu'il se passe en coulisse, avec un nouveau coauteur par exemple, c'est totalement différent et seul les auteurs doivent avoir accès à cette info. Comme dit au dessus, cette info se trouvera dans la liste affichée sur la version brouillon (car cette version n'est pas liée à un publishedContent) et dans la liste des commits.

Donc d'un point de vue "UX" on a "l'impression" que les auteurs sont "historisés" mais leur évolution ne l'est que lorsque le contenu est publié. Je pense donc vraiment qu'on est dans le cadre d'un simple ajout dans la bdd.

Par contre, perso, je ne considère pas ça comme un bug mais comme une évolution, à voir donc.

@artragis
Copy link
Member

J'aimerai savoir si la solution que je propose est validée histoire que je m'attèle au dev.

@pierre-24
Copy link
Member

Moi ça me semble tenir la route :) (j'espère qu'on est pas en train de réintroduire des choses en BDD qu'on voulais éviter).

@gustavi
Copy link
Contributor

gustavi commented Dec 5, 2015

Fixé dans #3199

@gustavi gustavi closed this as completed Dec 5, 2015
@firm1
Copy link
Contributor

firm1 commented Feb 19, 2016

Je réouvre cette issue. Car je suis tombé sur le tutoriel d'introduction à ruby. Dans sa version publique on voit Dominus en tant qu'auteur, alors que le tutoriel n'a jamais été publié avec lui comme auteur.

La dernière date de publication date de Juin 2015 et Dominus n'a été rajouté à la rédaction de ce tutoriel que très récemment.

@firm1 firm1 reopened this Feb 19, 2016
@artragis
Copy link
Member

Tu es prié de fermer l'issue : dominus a été ajouté car il était auteur
au moment où la maj a été faite.
Je ne peux pas deviner qui était auteur avant qu'on fasse la maj.

Le 19/02/2016 13:20, firm1 a écrit :

Je réouvre cette issue. Car je suis tombé sur le tutoriel
d'introduction à ruby
https://zestedesavoir.com/tutoriels/634/une-introduction-a-ruby.
Dans sa version publique on voit Dominus en tant qu'auteur, alors que
le tutoriel n'a jamais été publié avec lui comme auteur.

La dernière date de publication date de Juin 2015 et Dominus n'a été
rajouté à la rédaction de ce tutoriel que très récemment.


Reply to this email directly or view it on GitHub
#3078 (comment).

@pierre-24
Copy link
Member

Je suis d'accord avec @artragis sur celle là.

@firm1
Copy link
Contributor

firm1 commented Feb 19, 2016

Ah, effectivement. Je pensais qu'il y avait un historique des ajouts/sorties à la rédaction.

@artragis
Copy link
Member

Non, c'est d'ailleurs en ça que ma PR génère une évolution en terme de
fonctionnalité :p

Le 19/02/2016 13:27, firm1 a écrit :

Ah, effectivement. Je pensais qu'il y avait un historique des
ajouts/sorties à la rédaction.


Reply to this email directly or view it on GitHub
#3078 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django S-BUG Corrige un problème
Projects
None yet
Development

No branches or pull requests

7 participants