Development #14010
Possibilité d'abonner à une categorie à partir d'une liste d'adresses email fournie dans un CSV
0%
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a plus de 7 ans
- Sujet changé de Possibilité d'abonner à partir d'une liste d'adresses email à Possibilité d'abonner à une categorie à partir d'une liste d'adresses email fournie dans un CSV
- Echéance mis à 15 février 2017
- Assigné à mis à Serghei Mihai
- Priorité changé de Normal à Haut
Meyzieu revient sur cette demande (cf. #14673),
Je fixe l'échéance pour mi-février pour permettre l'abonnement des adresses mail depuis un CSV.
Mis à jour par Serghei Mihai il y a environ 7 ans
- Fichier 0001-import-subscribers-from-csv-14010.patch 0001-import-subscribers-from-csv-14010.patch ajouté
- Fichier 1.png 1.png ajouté
- Fichier 2.png 2.png ajouté
- Fichier 3.png 3.png ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
En reprenant quelques styles de combo pour le menu sandwich.
Et quelques captures d'écran pour avoir des avis.
Mis à jour par Frédéric Péters il y a environ 7 ans
Avant de faire des captures d'écran faut mettre à jour gadjo, parce que sinon tu paies ta tournée.
En reprenant quelques styles de combo pour le menu sandwich.
Très bien; mais l'objectif pour les trucs partagés est d'aller dans gadjo, pas d'être copiés/collés.
+ subscribers.append(row[0])
, pas besoin d'être dans le try/except.
+ if not row[0]:
, va violemment exploser sur une ligne vide dans le CSV.
+ url(r'^category/(?P<slug>[\w-]+)/import-subscriptions$', subscriptions_import,
, me semble qu'une convention django est de terminer les URL par des /.
+ <script src="{% static 'js/gadjo.js' %}"></script>
, il doit déjà y avoir gadjo.js présent depuis son template de base.
+ {{ form }}
, utiliser as_p, là clairement le manque d'espace entre le libellé et le widget fait mal aux yeux.
+ messages.info(self.request, _('%s subscriptions added') % new)
, utiliser ngettext. Et "%d new subscribers" plutôt ?
+ help_text=_('CSV file with email addresses on first column'))
C'est sympa de parler avec des phrases, et pour un format aussi peu défini que le CSV de préciser le charset et le délimiteur attendu.
Manquent des tests.
Mis à jour par Serghei Mihai il y a environ 7 ans
Frédéric Péters a écrit :
Très bien; mais l'objectif pour les trucs partagés est d'aller dans gadjo, pas d'être copiés/collés.
Ok, je fais un ticket séparé.
+ subscribers.append(row[0])
, pas besoin d'être dans le try/except.
Mon idée ici est d'intercepter le ValidationError levé par validate_email
et remonter plutôt à l'usager la ligne sur laquelle le mail n'est pas correct. Sinon ça dira juste "Invalid email".
Les autres remarques seront prises en charge dans le patch qui arrive.
Mis à jour par Serghei Mihai il y a environ 7 ans
- Lié à Development #14830: rajouter les styles pour le menu "sandwich" ajouté
Mis à jour par Frédéric Péters il y a environ 7 ans
+ subscribers.append(row0), pas besoin d'être dans le try/except.
Mon idée ici est d'intercepter le ValidationError levé par validate_email et remonter plutôt à l'usager la ligne sur laquelle le mail n'est pas correct. Sinon ça dira juste "Invalid email".
Je parle de la ligne subscribers.append(row[0])
.
Mis à jour par Serghei Mihai il y a environ 7 ans
Mis à jour par Frédéric Péters il y a environ 7 ans
print row
ngettext('%(new)d subscribers added', '%(new)d subscribers added', new)
: la première chaine est la version au singulier.
Mis à jour par Serghei Mihai il y a environ 7 ans
- Fichier 0001-import-subscribers-from-csv-14010.patch 0001-import-subscribers-from-csv-14010.patch ajouté
Corrigé, merci.
Mis à jour par Serghei Mihai il y a environ 7 ans
- Statut changé de En cours à Résolu (à déployer)
commit aa90f191a6aac0d8ed4076558dfc7d946c39c83e Author: Serghei Mihai <smihai@entrouvert.com> Date: Mon Jan 30 14:38:00 2017 +0100 import subscribers from csv (#14010)
import subscribers from csv (#14010)