Development #6842
Cellule de type "fil rss/atom"
0%
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 9 ans
- Bloqué par Development #7140: remontée de données ajouté
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Fichier 0001-misc-add-a-rss-atom-feed-cell-6842.patch 0001-misc-add-a-rss-atom-feed-cell-6842.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Voilà l'affaire de manière très basique, sans le grand plan général "remontée de données", pas vraiment nécessaire ici (contenu anonyme, cache ample et facile). C'est nécessaire maintenant pour le momo de Meaux (#7716).
Mis à jour par Thomas Noël il y a plus de 8 ans
A priori il manque le template.
Je passerais le cache à 10 minutes, histoire de permettre les demos, le temps que ça devienne paramétrable ou plus intelligent.
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Fichier 0001-misc-add-a-rss-atom-feed-cell-6842.patch 0001-misc-add-a-rss-atom-feed-cell-6842.patch ajouté
(template très très bête)
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
Je ne pense pas que ce soit une bonne idée d'utiliser l'URL directement comme clé pour le cache1, un coup de hashlib.md5 m'irait.
[1]: https://docs.djangoproject.com/en/1.8/topics/cache/#cache-key-warnings « Memcached, the most commonly-used production cache backend, does not allow cache keys longer than 250 characters or containing whitespace or control characters, and using such keys will cause an exception. »
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Fichier 0001-misc-add-a-rss-atom-feed-cell-6842.patch 0001-misc-add-a-rss-atom-feed-cell-6842.patch ajouté
Merci, modification apportée.
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
Hmm je ne vois pas bien comment le feed_content sorti du cache sera utilisé en lieu et place de l'appel à requests.get manque un if feed_content is None: quelque part.
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Fichier 0001-misc-add-a-rss-atom-feed-cell-6842.patch 0001-misc-add-a-rss-atom-feed-cell-6842.patch ajouté
Oulala.
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
Ack (mais je préférais quand la mécanique du cache n'était pas mélangé dans le code de rendu puis d'accès aux données, cf. blurp, quand on va vouloir utiliser le cache même après 10 minutes si le flux RSS ne répond pas ça va encore compliquer ce code).
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Statut changé de En cours à Résolu (à déployer)
commit be513d216603675f5a5359b2b0cb9d5591d2a299
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Mon Aug 24 14:32:06 2015 +0200
misc: add a rss/atom feed cell (#6842)
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: add a rss/atom feed cell (#6842)