Produits Entr'ouvert » Corbo

Tu peux décrire l'idée ?

C'est une idée en amont de celle du #13358.

On n'a pas de page de visualisation de l'annonce. juste l'édition dans ckeditor. Il est donc utile de pouvoir voir l'integralité de l'annonce sur une page à part.

Ok, c'est pas mal d'exposer le fil des idées. Donc dans #13358 je parle d'une page "DetailVew" pour les annonces, en supposant qu'elle existe. Mais elle n'existe pas. Doit-elle exister ? Peut-être que les actions sur une annonce elles auraient leur place dans la liste des annonces. Discussion zappée.

Page de visualisation d'une annonce, donc. N'appelons pas ça "prévisualisation", c'est une page qui reprend les infos de l'annonce, son titre, ses dates de publication ou dépublication, l'info comme quoi elle a déjà été envoyée ou pas, etc.

Et si c'est une annonce destinée à être envoyée par mail, pourquoi pas une prévisualisation du mail envoyé, mais c'est pour moi tout à fait différent. (par exemple il faudrait prendre en compte le template utilisé pour l'envoi, pas juste afficher .text, i.e. le corbo.announce.html avec le content et le unsubscribe_link_placeholder dans le context, de la méthode send()).

messages.info(self.request, _('Announce successfully updated'))

Communément on n'affiche pas de notif disant que tout s'est passé correctement; de plus comme l'usager revient ici sur une page où il peut voir l'annonce, il peut constater lui-même que c'est le cas.

{% for b in broadcasts %}

Également inclure le nombre de diffusions qu'on a comme info dans l'objet ?

Yep, je refais le patch.

delivers_number → delivery_count ?

ok

delivers_count dans le code :/

Ok, un petit bloc à droite avec les infos (cf. capture jointe)

Très clairement un peu plus large et les dates pourraient tenir. Ou alors formatter différemment, avoir libellé et valeur sur deux lignes distinctes. Là c'est dommage avec tout l'espace dont on dispose de couper la date/heure.

Delivered at {{ delivered }} to {{ delivers }} destinations

Ça passera sans doute une fois traduit en français mais "Delivered" laisse penser à une assurance comme quoi le message est bien arrivé, je préférerais être frileux et plutôt avoir "Sent".

Sur une de mes instances, j'obtiens ça à la migration :

  File "/home/fred/src/eo/venv/local/lib/python2.7/site-packages/django/db/backends/base/schema.py", line 440, in remove_field
    self.execute(sql)
  File "/home/fred/src/eo/venv/local/lib/python2.7/site-packages/django/db/backends/base/schema.py", line 111, in execute
    cursor.execute(sql, params)
  File "/home/fred/src/eo/venv/local/lib/python2.7/site-packages/django/db/backends/utils.py", line 79, in execute
    return super(CursorDebugWrapper, self).execute(sql, params)
  File "/home/fred/src/eo/venv/local/lib/python2.7/site-packages/django/db/backends/utils.py", line 64, in execute
    return self.cursor.execute(sql, params)
  File "/home/fred/src/eo/venv/local/lib/python2.7/site-packages/django/db/utils.py", line 98, in __exit__
    six.reraise(dj_exc_type, dj_exc_value, traceback)
  File "/home/fred/src/eo/venv/local/lib/python2.7/site-packages/django/db/backends/utils.py", line 64, in execute
    return self.cursor.execute(sql, params)
django.db.utils.OperationalError: ERREUR:  ne peut pas exécuter ALTER TABLE « corbo_broadcast » car il reste des événements sur les triggers

C'est expliqué http://stackoverflow.com/a/39541048 , le patch qui arrange les choses :

--- a/corbo/migrations/0010_broadcast_delivery_count.py
+++ b/corbo/migrations/0010_broadcast_delivery_count.py
@@ -25,6 +25,8 @@ class Migration(migrations.Migration):
     ]

     operations = [
+        migrations.RunSQL('SET CONSTRAINTS ALL IMMEDIATE',
+            reverse_sql=migrations.RunSQL.noop),
         migrations.AddField(
             model_name='broadcast',
             name='delivery_count',
@@ -35,4 +37,6 @@ class Migration(migrations.Migration):
             model_name='broadcast',
             name='result',
         ),
+        migrations.RunSQL(migrations.RunSQL.noop,
+            reverse_sql='SET CONSTRAINTS ALL IMMEDIATE'),
     ]

Ok.

"Évidemment" ça fait foirer la migration avec sqlite :/