search

ChantyTaguan / zds-site

Dépot ZDS
Other
0 stars 0 forks source link

Follow a forum, tag and topic #20

Closed GerardPaligot closed 9 years ago

GerardPaligot commented 9 years ago

Voilà, voilà. :)

ChantyTaguan commented 9 years ago

Ah ben j'étais précisément en train de retester l'abonnement à un tag ! Je regarde ça.

GerardPaligot commented 9 years ago

Tu remarqueras que je ne fais plus qu'une fois les requêtes pour savoir si l'utilisateur a déjà souscrit ou non ! :)

ChantyTaguan commented 9 years ago

Une fois pour l'abonnement, une fois pour les emails quand même. Non ? Pas moyen de réduire encore à une seule requête ? Genre au lieu de renvoyer un boolean, on renvoie l'abonnement même. Ou un mini dico avec active: True/False, by_email: True/False

GerardPaligot commented 9 years ago

Une fois pour l'abonnement, une fois pour les emails quand même. Non ? Pas moyen de réduire encore à une seule requête ? Genre au lieu de renvoyer un boolean, on renvoie l'abonnement même. Ou un mini dico avec active: True/False, by_email: True/False

Ou merger les 2 templates tag en renvoyer la subscription directement ?

ChantyTaguan commented 9 years ago

Oui c'est ça que je voulais dire par renvoyer l'abonnement même. Le dico c'est pour alléger mais c'est peut-être pas nécessaire.

GerardPaligot commented 9 years ago

Je vais tester ça alors.

GerardPaligot commented 9 years ago

Je m'occupe de cette PR ce soir si le Zest'Meeting n'est pas trop long.

Il faut que je pense aussi à corriger les messages suivis dans la sidebar.

GerardPaligot commented 9 years ago

Bon, j'ai renommé le templatetag. Par contre, je n'ai pas renvoyé la subscription dans les templatetags. J'avais commencé mais le contrepartie, c'est de devoir faire toutes les vérifications (vérifier si c'est None et/ou si c'est par e-mail ou pas) du côté des templates. Je suis donc moins convaincu de la pertinence de la chose parce que ça alourdit beaucoup les templates.

Qu'est ce que tu en penses ?

ChantyTaguan commented 9 years ago

Et si tu renvoies juste un dico ? Qui a deux booléens : active et by_email ?

GerardPaligot commented 9 years ago

Effectivement, j'avais oublié cette solution. Je test ça ce soir.

GerardPaligot commented 9 years ago

Les templates tags sont maintenants mergés. Si la PR te convient, je squash les commits et on merge.

ChantyTaguan commented 9 years ago

Ah merde, j'ai été trop vite, j'ai mergé. Tant pis ? Ou je revert ?

GerardPaligot commented 9 years ago

Ou alors je rebase ta branche principale, c'est comme tu veux :)

ChantyTaguan commented 9 years ago

Je n'ai aucun avis, mes compétences en git étant ce qu'elles sont... Le plus simple ?

GerardPaligot commented 9 years ago

Voilà, j'ai rebase ta branche :)

ChantyTaguan commented 9 years ago

Ah oui, ça a l'air simple comme ça :)

bon, histoire de pas refaire le coup de l'autre fois, je vire ma branche locale et je la re-checkout, c'est ça ?

GerardPaligot commented 9 years ago

Ca dépend, tu as déjà pull en local ? Si oui, alors tu dois effectivement créer une nouvelle branche. Si non, tu fais un simple git pull et ça devrait aller.

GerardPaligot commented 9 years ago

(En fait, je me demande si ça existe pas un git pull -f pour forcer la mise à jour d'une branche pas synchro avec sa branche distante)

ChantyTaguan commented 9 years ago

Bon, dans le doute, j'ai recréé ma branche