Développement #14010
Possibilité d'abonner à une categorie à partir d'une liste d'adresses email fournie dans un CSV
0%
Files
Related issues
Associated revisions
History
Updated by Serghei Mihai almost 8 years ago
- Subject changed from Possibilité d'abonner à partir d'une liste d'adresses email to Possibilité d'abonner à une categorie à partir d'une liste d'adresses email fournie dans un CSV
- Due date set to 15 February 2017
- Assignee set to Serghei Mihai
- Priority changed from Normal to 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.
Updated by Serghei Mihai almost 8 years ago
- File 0001-import-subscribers-from-csv-14010.patch 0001-import-subscribers-from-csv-14010.patch added
- File 1.png 1.png added
- File 2.png 2.png added
- File 3.png 3.png added
- Status changed from Nouveau to En cours
- Patch proposed changed from No to Yes
En reprenant quelques styles de combo pour le menu sandwich.
Et quelques captures d'écran pour avoir des avis.
Updated by Frédéric Péters almost 8 years ago
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.
Updated by Serghei Mihai almost 8 years ago
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.
Updated by Serghei Mihai almost 8 years ago
- Related to Développement #14830: rajouter les styles pour le menu "sandwich" added
Updated by Frédéric Péters almost 8 years ago
+ 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]).
Updated by Serghei Mihai almost 8 years ago
Updated by Frédéric Péters almost 8 years ago
print row
ngettext('%(new)d subscribers added', '%(new)d subscribers added', new) : la première chaine est la version au singulier.
Updated by Serghei Mihai almost 8 years ago
- File 0001-import-subscribers-from-csv-14010.patch 0001-import-subscribers-from-csv-14010.patch added
Corrigé, merci.
Updated by Serghei Mihai almost 8 years ago
- Status changed from En cours to 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)