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

Desinscription impossible #818

Closed
cgabard opened this issue Jun 9, 2014 · 63 comments
Closed

Desinscription impossible #818

cgabard opened this issue Jun 9, 2014 · 63 comments
Assignees
Labels
C-Back Concerne le back-end Django
Milestone

Comments

@cgabard
Copy link
Contributor

cgabard commented Jun 9, 2014

Il est important, avant la sortie de la version public de pouvoir se désinscrire. Rien ne le permet actuellement. Cette issue est plus restreinte que #668 , je ne m'interesse pas à l'avenir du contenu, mais juste à la possibilité pour un membre de se désinscrire.

@firm1
Copy link
Contributor

firm1 commented Jun 9, 2014

Techniquement on peut déjà de desinscrire en envoyant un mail au staff qui
fera le boulot.

Par contre je pense qu'il faut éviter de mettre en place une demie
desinscription. C'est a dire que la desisncription devra gérer les
problématiques de contenus présent sur le site.
Le 9 juin 2014 11:10, "Christophe Gabard" notifications@github.com a
écrit :

Il est important, avant la sortie de la version public de pouvoir se
désinscrire. Rien ne le permet actuellement. Cette issue est plus
restreinte que #668 #668
, je ne m'interesse pas à l'avenir du contenu, mais juste à la possibilité
pour un membre de se désinscrire.


Reply to this email directly or view it on GitHub
#818.

@cgabard
Copy link
Contributor Author

cgabard commented Jun 9, 2014

Je suis staff et je ne vois pas comment je peux le faire.
Le 9 juin 2014 12:12, "firm1" notifications@github.com a écrit :

Techniquement on peut déjà de desinscrire en envoyant un mail au staff qui
fera le boulot.

Par contre je pense qu'il faut éviter de mettre en place une demie
desinscription. C'est a dire que la desisncription devra gérer les
problématiques de contenus présent sur le site.
Le 9 juin 2014 11:10, "Christophe Gabard" notifications@github.com a
écrit :

Il est important, avant la sortie de la version public de pouvoir se
désinscrire. Rien ne le permet actuellement. Cette issue est plus
restreinte que #668 <
https://github.com/zestedesavoir/zds-site/issues/668>
, je ne m'interesse pas à l'avenir du contenu, mais juste à la
possibilité
pour un membre de se désinscrire.


Reply to this email directly or view it on GitHub
#818.


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

@Alex-D
Copy link
Contributor

Alex-D commented Jun 9, 2014

En passant par la BDD :D

@cgabard
Copy link
Contributor Author

cgabard commented Jun 9, 2014

Oui donc pas le staff, les admins c'est largement différents
Le 9 juin 2014 12:30, "Alexandre Demode" notifications@github.com a écrit
:

En passant par la BDD :D


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

@ShigeruM
Copy link
Contributor

ShigeruM commented Jun 9, 2014

Que faire du contenu des membres voulant se désinscrire ? Ce serait quelque chose comme :

  • les posts sur le forum sont anonymisés ;
  • les MP sont supprimés (le membre est retiré des MP et ses posts sont anonymisés) ;
  • les tutos et articles sont attribués à un compte particulier (mascotte ?) mais le nom de l'auteur reste apparant
  • etc ?

@Coy0te
Copy link
Contributor

Coy0te commented Jun 10, 2014

@ShigeruM : ton message a plutôt sa place dans l'issue #668.

Dans cette issue, je pense que l'important c'est de statuer si on a un moyen (quelconque) de désinscrire quelqu'un. Si oui, et qu'il est expliqué quelque part sur le site, alors l'issue est à clore.

@firm1
Copy link
Contributor

firm1 commented Jun 11, 2014

Dans cette issue, je pense que l'important c'est de statuer si on a un moyen (quelconque)

Actuellement pour désinscrire quelqu'un il suffit de faire la demande a quelqu'un qui a les pouvoirs et il le fera manuellement (le renommer anonyme et changer son mot de passe et son mail).

Mais l'issue reste ouverte pour débattre de l'automatisation de cette désinscription et des traitements à effectuer.

@geoffreyc
Copy link
Contributor

Outre l'aspect technique de la chose, je pense qu'il faudrait laisser le
choix a l'utilisateur de ce qu'il veut laisser sur le site ou non. Une
série de "checkboxes" du genre

Effacer mes tutoriels? [ ]
Effacer mes articles? [ ]
Effacer mes tutoriels? [ ]
Effacer mes messages? [ ]
Effacer mes commentaires? [ ]

Dans les cas des tutos/articles, on les effaces complètement. Dans le cas
messages/commentaires, on fait un mass update sur le contenu de tout les
messages du membre pour les effacer/remplacer par un texte indiquant
pourquoi il n'y a plus de contenu.

2014-06-11 15:21 GMT+01:00 firm1 notifications@github.com:

Dans cette issue, je pense que l'important c'est de statuer si on a un
moyen (quelconque)

Actuellement pour désinscrire quelqu'un il suffit de faire la demande a
quelqu'un qui a les pouvoirs et il le fera manuellement (le renommer
anonyme et changer son mot de passe et son mail).

Mais l'issue reste ouverte pour débattre de l'automatisation de cette
désinscription et des traitements à effectuer.


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

@cgabard
Copy link
Contributor Author

cgabard commented Jun 11, 2014

Pour les tutos/articles, pourquoi pas, plutot, les masquer uniquement
(dévalider + status particulier) puis laissé le soit aux valido de voir ?
Car techniquement ils seront en CC-BY... et donc reprenable si on en a
envie, non ?

Christophe.

2014-06-11 16:29 GMT+02:00 geoffreyc notifications@github.com:

Outre l'aspect technique de la chose, je pense qu'il faudrait laisser le
choix a l'utilisateur de ce qu'il veut laisser sur le site ou non. Une
série de "checkboxes" du genre

Effacer mes tutoriels? [ ]
Effacer mes articles? [ ]
Effacer mes tutoriels? [ ]
Effacer mes messages? [ ]
Effacer mes commentaires? [ ]

Dans les cas des tutos/articles, on les effaces complètement. Dans le cas
messages/commentaires, on fait un mass update sur le contenu de tout les
messages du membre pour les effacer/remplacer par un texte indiquant
pourquoi il n'y a plus de contenu.

2014-06-11 15:21 GMT+01:00 firm1 notifications@github.com:

Dans cette issue, je pense que l'important c'est de statuer si on a un
moyen (quelconque)

Actuellement pour désinscrire quelqu'un il suffit de faire la demande a
quelqu'un qui a les pouvoirs et il le fera manuellement (le renommer
anonyme et changer son mot de passe et son mail).

Mais l'issue reste ouverte pour débattre de l'automatisation de cette
désinscription et des traitements à effectuer.


Reply to this email directly or view it on GitHub
<
https://github.com/zestedesavoir/zds-site/issues/818#issuecomment-45747591>

.


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

@Coy0te
Copy link
Contributor

Coy0te commented Jun 11, 2014

@geoffreyc : je trouve assez improbable le fait de donner le choix au membre de supprimer ses messages/commentaires. L'anonymisation est le seul choix viable si on ne veut pas rendre le forum inutilisable.

@SpaceFox
Copy link
Member

Je ne suis pas d'accord avec toi, @geoffreyc :

  • Normalement les tutos et articles sont en CC. Donc le départ du membre et le retrait des tutos sont deux opérations différentes, la seconde est un pur problème éditorial.
  • Les messages et commentaires sont à conserver pour que le forum reste lisible.

@firm1
Copy link
Contributor

firm1 commented Jun 11, 2014

Moi je vois les options suivantes :

  • Tutoriels
    • Effacer : ici on les dépublie si l'auteur est le seul auteur, et on le vire de la liste des co-auteurs s'il y'a d'autres personnes
  • Articles
    • Effacer : ici on les dépublie si l'auteur est le seul auteur, et on le vire de la liste des co-auteurs s'il y'a d'autres personnes
  • Messages/Commentaires :
    • cacher ses messages/commentaires : on cache tous les messages de l'utilisateur
    • anonymiser les messages/commentaires : on anonymise

@geoffreyc
Copy link
Contributor

CC: spacefox

Niveau tuto/article, je comprend bien qu'ils sont en CC, mais veut on
vraiment donner l'image du "tanpis, on garde tes contributions, meme si
t'es pas content, parce que legalement on a le droit!" ? Perso je
préférerais laisser le choix au membre de si il veut le laisser ou pas.

pour les messages, "techniquement" (légalement?) si un membre en fait la
demande, on se doit de virer toutes ses contributions (mettre le message en
"anonyme" n'a pas vraiment l'air d'etre suffisant pour la CNIL). Ducoup je
vois pas de soucis a le proposer. Surtout qu'a l’époque sur le sdz on
l'avait vue plein de fois: un coup de script et les membres pouvaient
editer tous leurs messages pour les vider.

2014-06-11 15:36 GMT+01:00 Coyote notifications@github.com:

@geoffreyc https://github.com/geoffreyc : je trouve assez improbable le
fait de donner le choix au membre de supprimer ses messages/commentaires.
L'anonymisation est le seul choix viable si on ne veut pas rendre le forum
inutilisable.


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

@SpaceFox
Copy link
Member

OK pour laisser le membre demander la dé-validation de ses tutos / articles, mais pas pour coupler ça automatiquement avec la désinscription.

Souvent les désinscrtiptions sont du ragequit.
D'autre part, si tu es le seul à utiliser ton compte, ton tuto est normalement lu par plein de gens. Qui sont peut-être en train de le lire. Et si un tuto disparaît brutalement sans préavis, c'est aussi mauvais en terme d'image.

Pour moi la suppression des tutos doit être possible, mais c'est un vrai processus éditorial, dans lequel il doit y avoir une discussion avec les validos sur les raisons de la dévalidation et ce qu'on peut faire du tuto, pour respecter la volonté de l'auteur et le bien des lecteurs.

mettre le message en "anonyme" n'a pas vraiment l'air d'etre suffisant pour la CNIL

Ça je demande une source. Un post de forum ou un commentaire, à moins d'être particulièrement long et compliqué, n'est même pas considéré comme une œuvre de l'esprit.

Par contre il faut que l'anonymisation soit réelle, en particulier qu'on ne puisse pas différentier les messages de 2 anonymes différents.

cacher ses messages/commentaires : on cache tous les messages de l'utilisateur

Ça par contre pour moi c'est juste hors de question, parce que ça va rendre les forums où le membre a posté inutilisables.

@Alex-D Alex-D removed the Front label Jun 13, 2014
@ArnaudCalmettes ArnaudCalmettes added this to the Bêta publique milestone Jun 13, 2014
@ArnaudCalmettes
Copy link
Contributor

Est-ce qu'on peut encore considérer que cette issue est "bloquante bêta publique" si les utilisateurs ont quand même moyen de demander la suppression de leur compte à un admin ?

Pour moi, c'est au pire "bloquant V1", et encore ça tient limite de la feature ("désinscription automatique") qu'on peut placer en Post-V1-mais-faudra-pas-trop-trainer-non-plus. Vous en pensez quoi ?

@cgabard
Copy link
Contributor Author

cgabard commented Jun 14, 2014

Oui moi aussi. Tant qu'on a un moyen de le faire, ce n'est pas bloquant
Le 14 juin 2014 03:21, "ArnaudCalmettes" notifications@github.com a écrit
:

Est-ce qu'on peut encore considérer que cette issue est "bloquante bêta
publique" si les utilisateurs ont quand même moyen de demander la
suppression de son compte à un admin ?

Pour moi, c'est au pire "bloquant V1", et encore ça tient limite de la
feature ("désinscription automatique") qu'on peut placer en
Post-V1-mais-faudra-pas-trop-trainer-non-plus. Vous en pensez quoi ?


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

@ArnaudCalmettes
Copy link
Contributor

OK, dans ce cas, je passe l'issue en "bloquant V1". On avisera après la bêta.

@SpaceFox
Copy link
Member

SpaceFox commented Jul 9, 2014

Je propose qu'en v1 on indique dans le profil que pour l'instant pour se désinscrire il faut contacter un staffeux, et que qu'on fasse la vraie procédure au calme peu après la v1.

@Alex-D
Copy link
Contributor

Alex-D commented Jul 9, 2014

il faut contacter un staffeux

On peut s'en contenter tant que c'est gérable (donc bien après la v1 je pense). Non ? Enfin je pense que c'est quelque chose d'assez complexe à implémenter (faut rien oublier) et qui n'est pas super important face à d'autres modules cruciaux.

@pierre-24
Copy link
Member

On peut toujours en discuter jeudi !

@SpaceFox
Copy link
Member

On commence à avoir pas mal de PR en attente sur dev. Du coup pour faire avancer les choses et comme c'est le dernier ticket, je propose de sortir cette issue de la v1.0 et de la faire dès que possible.

C'est bon pour tout le monde ?

@artragis
Copy link
Member

ça me va ^^ ! Faudra le dire sur un article du site qu'on passe en V1 stable.

@Eskimon
Copy link
Member

Eskimon commented Aug 26, 2014

Mouai, moyen quand meme de retirer une demande légale pour nous arranger... A mon sens la v1 ne devrait pas etre V1 sans cette fonction...

@SpaceFox
Copy link
Member

Pour moi la problématique est qu'on a plus que ce ticket dans la pile, qu'il n'est pas prêt d'être fait (dans le sens où on a même pas de PR pour ça, donc qu'on ne l'aura pas mergable avant au moins ce WE) et donc qu'il devient complètement bloquant pour "sortir" la v1.0 (je détaille après ce que je veux dire par là).

Aujourd'hui on ne peut rien faire de propre sur dev tant qu'on aura pas mergé master dedans et donc qu'on aura sorti la v1.0. Or, on a beaucoup de corrections / évolutions faciles qui peuvent être faites rapidement, et on a déjà 6 PR en attente de merge. Le "bloquant" n'est donc pas une simple vue de l'esprit : on a déjà des actions qui attendent cette sortie.

D'autre part, l'obligation légale en question est de pouvoir supprimer un utilisateur (ce qui n'est pas "se désinscrire soi-même" à ma connaissance, toute source sérieuse est la bienvenue). Aujourd'hui si on rencontre le cas, on peut le faire manuellement (sauf erreur de ma part). L'option existe, mais elle demande de passer par le staff ; il manque sans doute une phrase pour écrire ça dans le profil avec un lien vers le contact et un autre vers cette issue.
Donc en repoussant la sortie de la fonctionnalité "se désinscrire", on ne retire aucune demande légale.

Enfin, fonctionnellement on est en v1.0 depuis la sortie. Cette v1.0 technique n'était là que pour bloquer le processus de développement le temps d'avoir une base suffisamment stable. Or à moins que cette issue ait des implications que je ne maîtrise pas, elle n'est pas nécessaire à la stabilité du socle technique et n'a été mise en v1.0 que parce qu'elle est urgente. Et cette urgence ne nécessite pas de faire attendre tout le reste du développement.

Pour moi les solutions que je vois sont :

  1. On merge master dans dev pour pouvoir faire avancer dev le temps de finir cette issue sur master. Et on va avoir plein de problèmes parce que les 2 branches vont très vite diverger (6 PR au moment où j'écris ces lignes).
  2. On continue à bloquer tout développement tant que cette issue n'est pas corrigée, soit jusqu'au WE prochain + le temps de QA au moins. Ici toute PR supplémentaire sur dev accroit le risque de conflits au moment de les merger.
  3. On considère qu'on a aucun besoin d'attendre et on fait avancer le schmilblick le plus efficacement possible en sortant cette issue de la v1.0 (moyennant une phrase dans le profil si besoin), et on sort une v1.1 dès que cette fonctionnalité est prête.

@Eskimon
Copy link
Member

Eskimon commented Aug 26, 2014

Ou d, la réponse d ;)

Perso la solution 3 me semble une "alternative minimum" viable. Un truc genre

"Pour vous désinscrire, merci de contacter xxx ou yyy (ou Clem ?). Votre demande sera alors effective dans un delai de 48h."

Ca me semble le minimum syndicale de proposer et expliquer la démarche a l'utilisateur. Une solution automatique sera bien sur la bienvenu pour éviter la charge de travail aux super-utilisateurs.

@Coy0te
Copy link
Contributor

Coy0te commented Aug 26, 2014

Il me semble qu'on avait déjà proposé ça comme truc temporaire avec Alex,
et qu'à l'époque ça avait été acquiescé. Si oui, go go go :)

Le 26 août 2014 14:14, Eskimon notifications@github.com a écrit :

Ou d, la réponse d ;)

Perso la solution 3 me semble une "alternative minimum" viable. Un truc
genre

"Pour vous désinscrire, merci de contacter xxx ou yyy (ou Clem ?). Votre
demande sera alors effective dans un delai de 48h."
Ca me semble le minimum syndicale de proposer et expliquer la démarche a
l'utilisateur. Une solution automatique sera bien sur la bienvenu pour
éviter la charge de travail aux super-utilisateurs.


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

@firm1
Copy link
Contributor

firm1 commented Aug 26, 2014

D'autre part, l'obligation légale en question est de pouvoir supprimer un utilisateur [...]. Aujourd'hui si on rencontre le cas, on peut le faire manuellement (sauf erreur de ma part). L'option existe, mais elle demande de passer par le staff

Je crois que l'option n'existe malheuresement pas. Pour supprimer un utilisateur aujourd'hui il faut supprimer tous ses posts (cascading toussa). Ce qui me parait illogique.

Donc, je pense qu'avant de décaler cette issue, il faut trouver une solution de contournement au problème actuel.

@SpaceFox SpaceFox assigned artragis and unassigned gustavi Sep 1, 2014
@SpaceFox SpaceFox modified the milestones: Version v1.x, Version 1.0, Version 1.1 Sep 1, 2014
@Alex-D Alex-D mentioned this issue Sep 7, 2014
@Alex-D
Copy link
Contributor

Alex-D commented Sep 24, 2014

Mergé par @SpaceFox s'en est donc terminé !

Il faudra être vigilent lors de la préprod et rouvrir en cas de besoin.

@Alex-D Alex-D closed this as completed Sep 24, 2014
@SpaceFox SpaceFox modified the milestone: Version 1.1 Sep 24, 2014
@SpaceFox
Copy link
Member

J'ai oublié de préciser : merci de ne pas toucher ni aux tickets, ni à la milestone.

Je repasse sur tout pour avoir quelque chose de propre, donc ça va être très bizarre les minutes qui viennent. C'est normal et voulu.

@SpaceFox SpaceFox added this to the Version 1.1 milestone Sep 24, 2014
@Eskimon
Copy link
Member

Eskimon commented Sep 24, 2014

En tout cas \o/ pour ce ticket !

@artragis
Copy link
Member

Il faudra que pendant la période de release quelqu'un vérifie que les TU
testent tous les cas.
Actuellement, je n'ai pas eu le temps de coder le test qui check que le
topic de la béta est bien altéré.
Le 24/09/2014 20:45, Eskimon a écrit :

En tout cas \o/ pour ce ticket !


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

@jcval
Copy link

jcval commented Oct 6, 2014

Petit fix ortho à faire :

"Parmi" (modale de la dernière chance) ne prend pas de "s"

@Eskimon Eskimon mentioned this issue Oct 6, 2014
@Alex-D
Copy link
Contributor

Alex-D commented Oct 6, 2014

Voilà qui est corrigé :) Merci @Eskimon !

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
Projects
None yet
Development

No branches or pull requests