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

Pouvoir (ne plus) suivre un article ou un tutoriel #2025

Closed
Situphen opened this issue Jan 1, 2015 · 13 comments
Closed

Pouvoir (ne plus) suivre un article ou un tutoriel #2025

Situphen opened this issue Jan 1, 2015 · 13 comments
Assignees
Labels
C-Back Concerne le back-end Django

Comments

@Situphen
Copy link
Member

Situphen commented Jan 1, 2015

Reporté par WinXaito sur le forum :

je ne sais pas si ça a déjà été dit, (J'ai recherché un petit coup mais n'ai rien trouvé), je n'ai pas vu la possibilité de ne plus suivre un "Article". Car j'ai remarqué qu'à chaque fois qu'un commentaire est dit dans l'article, j'en suis notifié (Pour autant que j'aie mis un commentaire), mais l'article ne fait pas partie des "Sujet suivi".

@Situphen Situphen changed the title Pour (ne plus) suivre un article ou un tutoriel Pouvoir (ne plus) suivre un article ou un tutoriel Jan 1, 2015
@gustavi gustavi added S-BUG Corrige un problème C-Back Concerne le back-end Django labels Jan 1, 2015
@gustavi gustavi added this to the "Futur proche" (v1.x) milestone Jan 1, 2015
@ChantyTaguan
Copy link
Contributor

Hello,

J'aurais bien envie de m'occuper de ceci. Après lecture du code, je vois qu'on a pas de structure dédiée pour le suivi des articles et des tutos. On regarde simplement si on a écrit un commentaire sur l'article/le tuto et si y a des réponses non lues. Pour les forums par contre, y a un modèle dédié.

Du coup, c'est l'occasion de peut-être penser encore plus loin et faire des notifs complètement générales (liées à un sujet du forum, un article, un tuto, mais éventuellement à d'autres choses, genre les futures tribunes libres).
Est-ce qu'il y a déjà eu une discussion à ce sujet ? Est-ce que ça mérite carrément une ZEP (pas l'impression) ?
Est-ce qu'éventuellement y a une manière classique d'implémenter ce genre de truc avec Django/python ?

@Eskimon
Copy link
Contributor

Eskimon commented Jan 7, 2015

Le truc c'est que le code actuel concernant les reactions sur articles/tuto est loin d'etre propre. Il est duplique de partout et c'est pas la joie. Normalement cette duplication va degager avec la ZEP-12 en cours de dev', mais je sais pas si ca un impact "factorisant" vis a vis des réactions du forum... (ping @pierre-24 && @gustavi || @artragis ^^)

@artragis
Copy link
Member

artragis commented Jan 7, 2015

A priori non. Il faudra implémenter ça une fois la zep 12 développée.
Le 07/01/2015 08:07, Eskimon a écrit :

Le truc c'est que le code actuel concernant les reactions sur
articles/tuto est loin d'etre propre. Il est duplique de partout et
c'est pas la joie. Normalement cette duplication va degager avec la
ZEP-12 en cours de dev', mais je sais pas si ca un impact
"factorisant" vis a vis des réactions du forum... (ping @pierre-24
https://github.com/pierre-24 && @gustavi
https://github.com/gustavi || @artragis
https://github.com/artragis ^^)


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

@pierre-24
Copy link
Member

On va pas tout bloquer parce que ZEP-12 non plus (je voudrais pas être pessimiste, mais ça va prendre un peu de temps à dev, la ZEP-12). Donc on peut très clairement lancer une ZEP, et ça va pas être bloquant, à mon sens ;)

@artragis
Copy link
Member

artragis commented Jan 7, 2015

oui, c'est à vous de vvoir. mais c'est pas la zep 12 qui va dev la
fonctionnalité.
Le 07/01/2015 08:30, Pierre Beaujean a écrit :

On va pas tout bloquer parce que ZEP-12 non plus (je voudrais pas être
pessimiste, mais ça va prendre un peu de temps à dev, la ZEP-12). Donc
on peut très clairement lancer une ZEP, et ça va pas être bloquant, à
mon sens ;)


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

@Eskimon
Copy link
Contributor

Eskimon commented Apr 16, 2015

Je met ici un mot cle : ZEP-12 (pour me rappeler de revenir ici) . @pierre-24 @artragis vous confirmez que ca sera dedans ?

@artragis
Copy link
Member

autoquote

oui, c'est à vous de vvoir. mais c'est pas la zep 12 qui va dev la
fonctionnalité.

2015-04-16 14:55 GMT+02:00 Eskimon notifications@github.com:

Je met ici un mot cle : ZEP-12 (pour me rappeler de revenir ici) .
@pierre-24 https://github.com/pierre-24 @artragis
https://github.com/artragis vous confirmez que ca sera dedans ?


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

@Eskimon
Copy link
Contributor

Eskimon commented Apr 16, 2015

ok :) Mais comme un peu plus tot on parlais du suivi auto pour l'auteur, est-ce que ca ne pourrait pas s'en inspirer ?

@artragis
Copy link
Member

Dans le futur il faudra s'en inspirer, pour l'instant c'est un peu...
difficile.

2015-04-16 15:13 GMT+02:00 Eskimon notifications@github.com:

ok :) Mais comme un peu plus tot on parlais du suivi auto pour l'auteur,
est-ce que ca ne pourrait pas s'en inspirer ?


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

@artragis
Copy link
Member

Au pire à voir avec celui qui va dev l'inscription de l'auteur. #Pasmoi

Le 16 avril 2015 15:16, francois dambrine artragis@gmail.com a écrit :

Dans le futur il faudra s'en inspirer, pour l'instant c'est un peu...
difficile.

2015-04-16 15:13 GMT+02:00 Eskimon notifications@github.com:

ok :) Mais comme un peu plus tot on parlais du suivi auto pour l'auteur,
est-ce que ca ne pourrait pas s'en inspirer ?


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

@Eskimon Eskimon removed this from the "Futur proche" (v1.x) milestone Apr 21, 2015
@Eskimon
Copy link
Contributor

Eskimon commented May 10, 2015

Je tag evolution, la feature n'existant pas elle ne peut pas être bugguée :-°

@Eskimon Eskimon added Evolution and removed S-BUG Corrige un problème labels May 10, 2015
@Situphen
Copy link
Member Author

Faudra se rappeler de fermer ici lors du merge de la ZEP 24 !

@GerardPaligot
Copy link
Member

@Situphen Toutafé. Je me l'assigne pour y penser.

@GerardPaligot GerardPaligot self-assigned this Feb 13, 2016
@gustavi gustavi closed this as completed in 8b771a8 Mar 5, 2016
Emeric54 pushed a commit to Emeric54/zds-site that referenced this issue Mar 20, 2016
Emeric54 pushed a commit to Emeric54/zds-site that referenced this issue Apr 2, 2016
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

7 participants