Projet

Général

Profil

Activité

Du 08 octobre 2016 au 06 novembre 2016

06 novembre 2016

17:45 Bug #13877: Trier les annonces par ordre chronologique (inverse)
Ack Thomas Noël
17:22 Bug #13877 (En cours): Trier les annonces par ordre chronologique (inverse)
Frédéric Péters
17:22 Bug #13877 (Fermé): Trier les annonces par ordre chronologique (inverse)
En voyant une catégorie avec pas mal d'annonces, ce qui peut arriver très rapidement si elle est créée comme étant l'... Frédéric Péters
17:29 Bug #13879 (En cours): ajustement style affichage de la date de publication
Frédéric Péters
17:28 Bug #13879 (Fermé): ajustement style affichage de la date de publication
Par rapport au titre de l'annonce, la date de publication est trop collée au bord de ligne; elle est aussi un peu tro... Frédéric Péters
17:26 Development #13878 (Fermé): Pagination des annonces
Quand elles sont nombreuses, ça pourrait être utile d'avoir une pagination. Frédéric Péters
17:21 Révision 3f6c77a1: manage: order announces chronologically (#13877)
Frédéric Péters
17:13 Bug #13876 (Fermé): poser les fichiers images sortis des fils dans un sous-répertoire
Pour le moment c'est directement sous /media/, ce qui n'est pas très élégant. Frédéric Péters

03 novembre 2016

18:18 Bug #13614 (Résolu (à déployer)): gueule de la page de désinscription
Fait.... Serghei Mihai
17:51 Bug #13614: gueule de la page de désinscription
Dans subscription_confirm_delete.html, taper {% trans 'Click on "Unsubscribe" to stop receiving emails to this addres... Frédéric Péters
17:39 Bug #13614: gueule de la page de désinscription
Je veux bien que ce patch soit accepté et avoir un retour des utilisateurs. Ensuite réflechir à l'option d'avoir la r... Serghei Mihai
16:28 Bug #13614: gueule de la page de désinscription
Est-ce que les modifications sur la vue dont tu parles, tu comptes les faire dans ce ticket et du coup j'attends le p... Frédéric Péters
18:17 Révision c7da4c3f: re-style unsubscription pages (#13614)
Serghei Mihai

31 octobre 2016

16:26 Bug #13450: considérer toutes les images dans le corps d'une annonce comme "distantes"
Premier patch refait avec la vérification du code de retour lors du téléchargement de l'image. Serghei Mihai
10:44 Bug #13450: considérer toutes les images dans le corps d'une annonce comme "distantes"
Thomas Noël a écrit :
> « image_name = os.path.basename(img.attrib['src']) » ça me semble un peu casse-gueule avec d...
Serghei Mihai
10:29 Bug #13450: considérer toutes les images dans le corps d'une annonce comme "distantes"
« image_name = os.path.basename(img.attrib['src']) » ça me semble un peu casse-gueule avec des src du genre https://m... Thomas Noël
10:27 Bug #13450: considérer toutes les images dans le corps d'une annonce comme "distantes"
Ce n'était pas le cas dans le code d'avant mais il faut en fait gérer le cas où requests.get() ne retourne pas une 200. Frédéric Péters

28 octobre 2016

17:32 Bug #13791 (Fermé): utiliser des slugs plutôt que des id dans les endpoints
Utiliser des id ça complique la migration de plateformes en plateformes. Frédéric Péters
17:29 Development #13284 (Résolu (à déployer)): fournir un point d'API d'abonnement compatible wcs
Josué Kouka
17:24 Development #13284: fournir un point d'API d'abonnement compatible wcs
Ok Serghei Mihai
17:26 Révision 77674a4e: add simple subscription API (#13284)
Josué Kouka
17:26 Révision c13b4c9b: misc: fix pep8 errors
Josué Kouka
16:51 Bug #13450: considérer toutes les images dans le corps d'une annonce comme "distantes"
D'abord un patch qui télécharge toutes les images "exterieures" lors de la création d'une annonce. Cela complète l'id... Serghei Mihai
11:59 Development #13785 (Résolu (à déployer)): Ajouter des permissions à l'API
... Serghei Mihai
11:55 Development #13785: Ajouter des permissions à l'API
ok :/ Frédéric Péters
11:39 Development #13785: Ajouter des permissions à l'API
Tous les points de l'API exigeront les permissions et donc ça me parait plus pertinent de le faire au niveau "global". Serghei Mihai
11:34 Development #13785: Ajouter des permissions à l'API
J'aime moins; c'est quoi la motivation pour écrire ça ainsi ? Frédéric Péters
11:24 Development #13785: Ajouter des permissions à l'API
Patch refait avec les permissions déclarés au niveau des settings. Serghei Mihai
11:21 Development #13785: Ajouter des permissions à l'API
ok. Frédéric Péters
11:13 Development #13785: Ajouter des permissions à l'API
My bad. Serghei Mihai
11:55 Development #13212: virer le "transport" "Accueil"
Ça n'existe pas les autoack. Frédéric Péters
11:34 Development #13212 (Résolu (à déployer)): virer le "transport" "Accueil"
Auto ack.... Serghei Mihai
11:33 Révision b84a6769: api: fix access permissions (#13785)
Serghei Mihai
11:33 Révision e147eff6: remove "homepage" transport (#13212)
Serghei Mihai

27 octobre 2016

18:25 Development #13785 (En cours): Ajouter des permissions à l'API
Josué Kouka
17:50 Development #13785 (Fermé): Ajouter des permissions à l'API
Josué Kouka
18:01 Development #13284: fournir un point d'API d'abonnement compatible wcs
Il y a des tests. Qui passent sans autorisations. Frédéric Péters
17:58 Development #13284: fournir un point d'API d'abonnement compatible wcs
Le contrôle d'accès à l'API est fait par Django Rest Framework. Serghei Mihai
17:53 Development #13284: fournir un point d'API d'abonnement compatible wcs
> Sans ça, ack ou pas ?
Bien sûr que non, avec un bug pareil ça ne peut pas être commité.
Frédéric Péters
17:51 Development #13284: fournir un point d'API d'abonnement compatible wcs
Frédéric Péters a écrit :
> La PEP8 ça concerne la syntaxe, pour moi ça ne peut pas contenir des suppressions d'impo...
Josué Kouka
17:45 Development #13284: fournir un point d'API d'abonnement compatible wcs
La PEP8 ça concerne la syntaxe, pour moi ça ne peut pas contenir des suppressions d'import ou de variables inutilisée... Frédéric Péters
17:34 Development #13284: fournir un point d'API d'abonnement compatible wcs
"testy" dans le message du premier commit.
Dans le patch tu fais aussi quelques corrections PEP8 dans le fichier @co...
Serghei Mihai

26 octobre 2016

17:51 Development #12919 (Résolu (à déployer)): création d'annonces depuis un flux RSS externe
Auto-ack:... Serghei Mihai

20 octobre 2016

13:41 Bug #13614 (En cours): gueule de la page de désinscription
Un début de refonte inspiré des captures en exemple.
Sur la page "unsubscription/done" il manque soit l'adresse mail...
Serghei Mihai
11:46 Development #13212: virer le "transport" "Accueil"
Le bon patch. Serghei Mihai
11:41 Bug #13596 (Résolu (à déployer)): Ne pas utiliser un lien sur "ici" pour le désabonnement
... Serghei Mihai
11:32 Bug #13596: Ne pas utiliser un lien sur "ici" pour le désabonnement
ack. Frédéric Péters
11:15 Bug #13596 (En cours): Ne pas utiliser un lien sur "ici" pour le désabonnement
Serghei Mihai
11:41 Révision 7cf80d56: create announces from external RSS feed (#12919)
Serghei Mihai
11:41 Révision 7e2730e4: don't use 'click here' text for unsubscribre link (#13596)
Serghei Mihai

19 octobre 2016

09:49 Bug #13659 (Résolu (à déployer)): supporter Django 1.8
... Serghei Mihai
09:48 Révision 3e52fbb3: setup.py: add support for django 1.8 (#13659)
Serghei Mihai

18 octobre 2016

21:09 Bug #13659: supporter Django 1.8
Ack. Benjamin Dauvergne
18:28 Bug #13659 (En cours): supporter Django 1.8
Serghei Mihai
18:28 Bug #13659 (Fermé): supporter Django 1.8
Même si le code tourne sous Django 1.8, le setup.py demande une version inférieure. Serghei Mihai

14 octobre 2016

18:38 Development #13212 (En cours): virer le "transport" "Accueil"
D'accord Serghei Mihai
18:20 Development #12919: création d'annonces depuis un flux RSS externe
Frédéric Péters a écrit :
> [...]
>
> Mmm, si jamais les attributs contenaient des URL relatives ? (peut-être pa...
Serghei Mihai
17:48 Development #12919: création d'annonces depuis un flux RSS externe
... Frédéric Péters
17:05 Development #12919: création d'annonces depuis un flux RSS externe
Patch à jour avec téléchargement/mise à jour des images des flux RSS afin de pouvoir les servir depuis l'instance de ... Serghei Mihai
18:16 Bug #13614 (Fermé): gueule de la page de désinscription
À travailler; en exemple l'écran de la newsletter envoyée par apsulis. Frédéric Péters
16:16 Bug #13450: considérer toutes les images dans le corps d'une annonce comme "distantes"
Ok, je vois. C'est pour l'affichage des images dans le flux RSS emis par corbo. Je refais le patch dans #12919 Serghei Mihai

13 octobre 2016

18:52 Development #13552 (Résolu (à déployer)): Ne pas recalculer le template du message à chaque envoi
... Serghei Mihai
18:47 Development #13552: Ne pas recalculer le template du message à chaque envoi
Ok, testé. Frédéric Péters
18:15 Development #13552: Ne pas recalculer le template du message à chaque envoi
Non, car le lien de desabonnement est fait par catégorie et non par annonce. Serghei Mihai
18:09 Development #13552: Ne pas recalculer le template du message à chaque envoi
Il manque pas un niveau d'indentation sur la dernière ligne ?
(et j'aurais plutôt fait ça avec un dictionnaire, as...
Frédéric Péters
18:04 Development #13552: Ne pas recalculer le template du message à chaque envoi
Ok Serghei Mihai
18:50 Bug #13596 (Fermé): Ne pas utiliser un lien sur "ici" pour le désabonnement
Ça fait plus de quinze ans qu'a été publié https://www.w3.org/QA/Tips/noClickHere
(pour une lecture plus récente, ...
Frédéric Péters
18:01 Révision 9841eeb4: compute email message body once (#13552)
Serghei Mihai

12 octobre 2016

16:59 Development #13552: Ne pas recalculer le template du message à chaque envoi
J'étendrais le test_unsubscription_link pour vérifier que les messages contiennent bien chacun des unsubscription_lin... Frédéric Péters
15:34 Development #13552: Ne pas recalculer le template du message à chaque envoi
Chez moi ils passent, mais je suis preneur d'un ack ou nack de qqun qui a appliqué le patch en local et qui l'a executé. Serghei Mihai
11:54 Development #13552: Ne pas recalculer le template du message à chaque envoi
Si ça passe les tests, ack. Benjamin Dauvergne
14:37 Bug #13450: considérer toutes les images dans le corps d'une annonce comme "distantes"
Je propose deux options, tu dis non aux deux ? Parce que pour les deux, l'intérêt c'est pouvoir servir les images ... Frédéric Péters
14:26 Bug #13450: considérer toutes les images dans le corps d'une annonce comme "distantes"
Je ne suis pas sûr de voir l'interet de récuperer les images chez nous et de les servir ensuite. Serghei Mihai

11 octobre 2016

17:17 Development #13552: Ne pas recalculer le template du message à chaque envoi
Patch à jour avec le placeholder défini directement dans le fichier modèle. Serghei Mihai
16:39 Development #13552 (En cours): Ne pas recalculer le template du message à chaque envoi
Serghei Mihai
16:39 Development #13552 (Fermé): Ne pas recalculer le template du message à chaque envoi
Au lieu de compiler le template pour chaque abonné en y inserant le lien de désabonnement, le corps du message doit ê... Serghei Mihai
16:54 Bug #13450: considérer toutes les images dans le corps d'une annonce comme "distantes"
> Je ne voudrais pas avoir à gérer des situations ou une image change mais pas son nom.
Alors la récupérer à chaqu...
Frédéric Péters
16:37 Bug #13450: considérer toutes les images dans le corps d'une annonce comme "distantes"
Frédéric Péters a écrit :
> Je trouverais plus chouette que corbo aspire les images des fils RSS pour les avoir en l...
Serghei Mihai
11:03 Bug #13450: considérer toutes les images dans le corps d'une annonce comme "distantes"
Je trouverais plus chouette que corbo aspire les images des fils RSS pour les avoir en local, ça permettrait ainsi d'... Frédéric Péters
11:57 Development #12919: création d'annonces depuis un flux RSS externe
Ok, merci.
Et au passage j'utilise le bon attribut @published@ et non @updated@.
Serghei Mihai
10:47 Development #12919: création d'annonces depuis un flux RSS externe
feedparser fait déjà le taf de lecture des dates. (https://pythonhosted.org/feedparser/date-parsing.html) Frédéric Péters

10 octobre 2016

10:08 Bug #13520 (Résolu (à déployer)): Supprimer la trad anglaise
... Frédéric Péters
09:43 Bug #13520: Supprimer la trad anglaise
Ok Serghei Mihai
10:08 Bug #13519 (Résolu (à déployer)): inclure le fichier VERSION dans le tar.gz
... Frédéric Péters
09:43 Bug #13519: inclure le fichier VERSION dans le tar.gz
Ack Serghei Mihai

09 octobre 2016

13:04 Bug #13520 (En cours): Supprimer la trad anglaise
Frédéric Péters
13:03 Bug #13520 (Fermé): Supprimer la trad anglaise
Il y a par erreur un .po pour l'anglais. Frédéric Péters
13:03 Révision 2f4bf293: misc: remove useless English .po file (#13520)
Frédéric Péters
13:01 Bug #13519 (En cours): inclure le fichier VERSION dans le tar.gz
Frédéric Péters
13:00 Bug #13519 (Fermé): inclure le fichier VERSION dans le tar.gz
Ce n'est pas le cas aujourd'hui et la bonne information ne se trouve ainsi pas donnée par pkg_resources (@corbo 0 (/u... Frédéric Péters
13:00 Révision 69819fea: build: ship VERSION file (#13519)
Frédéric Péters
 

Formats disponibles : Atom