Activité
Du 25 octobre 2016 au 23 novembre 2016
23 novembre 2016
- 18:03 Development #14005: Traduire le texte de désinscription
- → from django.utils.translation import activate et activate(settings.LANGUAGE_CODE).
- 18:00 Development #14005: Traduire le texte de désinscription
- Au fait, le texte est déjà traduit: http://git.entrouvert.org/corbo.git/tree/corbo/locale/fr/LC_MESSAGES/django.po#n1...
- 17:52 Development #14005: Traduire le texte de désinscription
- Correction; ça mérite d'aller dans la version à venir. (que je pensais déjà tagguée)
- 17:51 Development #14005: Traduire le texte de désinscription
- Par contre, l'objet premier du ticket, traduire la chaine, il reste d'actualité. (ça aurait mérité d'aller dans la n...
- 17:44 Bug #14062 (Résolu (à déployer)): tests cassés
- ...
- 17:41 Bug #14062: tests cassés
- Bon, ok, sans trop chercher à comprendre comment un rebase raté peut en arriver là, c'est dans l'utilisation de mailo...
- 17:24 Bug #14062: tests cassés
- Rebase foiré en local...
- 17:16 Bug #14062: tests cassés
- Le changement est causé par quoi ?
- 17:02 Bug #14062 (En cours): tests cassés
- 17:02 Bug #14062 (Fermé): tests cassés
- ...
- 17:02 Révision 8fcdf922: tests: fix inline images check test (#14062)
- 16:36 Révision 087922cc: tests: import missing mock module
- 16:29 Bug #13889 (Résolu (à déployer)): commande sync_external_feeds manquante
- @continue@ viré....
- 16:08 Bug #13889: commande sync_external_feeds manquante
- Sans le @continue@ inutile, ack.
- 16:28 Révision 1829923d: add command for external rss feed synchronization (#13889)
- 16:26 Development #14046 (Résolu (à déployer)): utiliser la fixture mailoutbox dans les tests
- ...
- 16:07 Development #14046: utiliser la fixture mailoutbox dans les tests
- ok.
- 16:05 Development #14046: utiliser la fixture mailoutbox dans les tests
- Yep.
- 12:38 Development #14046: utiliser la fixture mailoutbox dans les tests
- Il manque une modification au tox.ini pour préciser la version requise.
@assert len(mailoutbox)@ comparer à la tai... - 12:32 Development #14046 (En cours): utiliser la fixture mailoutbox dans les tests
- 16:21 Révision 048f0681: tests: use new 'mailoutbox' fixture (#14046)
- 09:28 Development #13284: fournir un point d'API d'abonnement compatible wcs
- Non.
- 08:19 Development #13284: fournir un point d'API d'abonnement compatible wcs
- est-ce que c'est testable en recette (voire déjà en prod) ?
22 novembre 2016
- 16:06 Bug #13889: commande sync_external_feeds manquante
- A priori c'est executé dans le contexte de chaque tenant, mais tu as raison, mieux vaut charger le logguer dans la mé...
- 12:23 Bug #13889: commande sync_external_feeds manquante
- Je connais mal cette affaire mais a priori, le logger, je ne le chargerais pas à l'import du module, sur l'idée que l...
- 14:46 Development #14005: Traduire le texte de désinscription
- Je pense important d'avoir une uniformité, que ça se configure une fois par toutes, que ça soit présent dans le modèl...
- 14:20 Development #14005: Traduire le texte de désinscription
- Pour l'instant c'est personalisable uniquement dans le template html dans lequel le message est inseré.
Il est tou... - 13:51 Development #14046 (Fermé): utiliser la fixture mailoutbox dans les tests
- Les tests échouent avec le message:...
21 novembre 2016
- 09:45 Development #12919: création d'annonces depuis un flux RSS externe
- Traité dans #13889.
20 novembre 2016
- 09:48 Development #12919 (En cours): création d'annonces depuis un flux RSS externe
- Et l'autoack passe à côté du fichier sync_external_feeds.py qui manque.
17 novembre 2016
- 16:52 Development #14010 (Fermé): Possibilité d'abonner à une categorie à partir d'une liste d'adresses email fournie dans un CSV
- 16:51 Development #14006: Ajouter la possibilité de visualiser l'annonce en ligne
- Ici je pointe #13286, direction que je préfère. Bien sûr entre l'assemblage et l'envoi, il y a visualisation et poss...
- 14:43 Development #14006 (Rejeté): Ajouter la possibilité de visualiser l'annonce en ligne
- Avant envoi pour vérification du résultat et après envoi pour insérer un lien : "voir en ligne" comme dans tout -bon ...
- 14:27 Development #14005 (Fermé): Traduire le texte de désinscription
- "Click here to unsubscribe from this newsletter. "
Au passage est-ce que ce texte est customisable par client ? (e...
16 novembre 2016
- 18:20 Bug #13993 (Résolu (à déployer)): debian : ne pas tenter de lancer les jobs cron quand le paquet est supprimé mais pas purgé
- ...
- 18:19 Bug #13993: debian : ne pas tenter de lancer les jobs cron quand le paquet est supprimé mais pas purgé
- Ack
- 18:11 Bug #13993 (En cours): debian : ne pas tenter de lancer les jobs cron quand le paquet est supprimé mais pas purgé
- 18:10 Bug #13993 (Fermé): debian : ne pas tenter de lancer les jobs cron quand le paquet est supprimé mais pas purgé
- Parce que sinon, #13992.
- 18:10 Révision 924aab53: debian: check for corbo-manage before running cron jobs (#13993)
15 novembre 2016
- 11:46 Bug #13450: considérer toutes les images dans le corps d'une annonce comme "distantes"
- Ok
- 11:07 Bug #13889: commande sync_external_feeds manquante
- Ok.
- 10:04 Bug #13889: commande sync_external_feeds manquante
- Au moins "logguer" l'erreur.
- 09:44 Bug #13889: commande sync_external_feeds manquante
- Ok, je passe toute exception levée.
Des cas comme flux RSS indisponibles doivent être gérés dans le @save@. - 09:43 Bug #13879 (Résolu (à déployer)): ajustement style affichage de la date de publication
- ...
- 09:42 Bug #13890 (Résolu (à déployer)): debian: destinataire des erreurs de crontab
- ...
- 09:42 Révision 8b380812: style: adjust padding and color of publication datetime (#13879)
- 09:41 Révision 8e0e5be0: debian: direct cronjob output to root (#13890)
14 novembre 2016
- 00:23 Bug #13879: ajustement style affichage de la date de publication
- Ack
- 00:18 Bug #13890: debian: destinataire des erreurs de crontab
- Ack
- 10:42 Bug #13890 (En cours): debian: destinataire des erreurs de crontab
- 16:47 Bug #13889: commande sync_external_feeds manquante
- timezone et CommandError importés pour rien.
Pas sûr qu'on ait envie de planter sur le premier fil indisponible. - 16:40 Bug #13889 (En cours): commande sync_external_feeds manquante
- 16:32 Bug #13450: considérer toutes les images dans le corps d'une annonce comme "distantes"
- Mettons une première entrée avec une première image, 2016/photo.jpeg. Le fichier n'existe pas, save() crée un fichie...
10 novembre 2016
07 novembre 2016
- 13:05 Bug #13890 (Fermé): debian: destinataire des erreurs de crontab
- Aujourd'hui les jobs sont envoyés à l'usager corbo local, qui n'existe pas, soit il faut en faire un alias vers root@...
- 13:00 Bug #13889 (Fermé): commande sync_external_feeds manquante
06 novembre 2016
- 17:45 Bug #13877: Trier les annonces par ordre chronologique (inverse)
- Ack
- 17:22 Bug #13877 (En cours): Trier les annonces par ordre chronologique (inverse)
- 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'...
- 17:29 Bug #13879 (En cours): ajustement style affichage de la date de publication
- 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...
- 17:26 Development #13878 (Fermé): Pagination des annonces
- Quand elles sont nombreuses, ça pourrait être utile d'avoir une pagination.
- 17:21 Révision 3f6c77a1: manage: order announces chronologically (#13877)
- 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.
03 novembre 2016
- 18:18 Bug #13614 (Résolu (à déployer)): gueule de la page de désinscription
- Fait....
- 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...
- 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...
- 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...
- 18:17 Révision c7da4c3f: re-style unsubscription pages (#13614)
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.
- 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... - 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...
- 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.
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.
- 17:29 Development #13284 (Résolu (à déployer)): fournir un point d'API d'abonnement compatible wcs
- 17:24 Development #13284: fournir un point d'API d'abonnement compatible wcs
- Ok
- 17:26 Révision 77674a4e: add simple subscription API (#13284)
- 17:26 Révision c13b4c9b: misc: fix pep8 errors
- 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...
- 11:59 Development #13785 (Résolu (à déployer)): Ajouter des permissions à l'API
- ...
- 11:55 Development #13785: Ajouter des permissions à l'API
- ok :/
- 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".
- 11:34 Development #13785: Ajouter des permissions à l'API
- J'aime moins; c'est quoi la motivation pour écrire ça ainsi ?
- 11:24 Development #13785: Ajouter des permissions à l'API
- Patch refait avec les permissions déclarés au niveau des settings.
- 11:21 Development #13785: Ajouter des permissions à l'API
- ok.
- 11:13 Development #13785: Ajouter des permissions à l'API
- My bad.
- 11:55 Development #13212: virer le "transport" "Accueil"
- Ça n'existe pas les autoack.
- 11:34 Development #13212 (Résolu (à déployer)): virer le "transport" "Accueil"
- Auto ack....
- 11:33 Révision b84a6769: api: fix access permissions (#13785)
- 11:33 Révision e147eff6: remove "homepage" transport (#13212)
27 octobre 2016
- 18:25 Development #13785 (En cours): Ajouter des permissions à l'API
- 17:50 Development #13785 (Fermé): Ajouter des permissions à l'API
- 18:01 Development #13284: fournir un point d'API d'abonnement compatible wcs
- Il y a des tests. Qui passent sans autorisations.
- 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.
- 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é. - 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... - 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...
- 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...
26 octobre 2016
Formats disponibles : Atom