Project

General

Profile

Development #33817

action notification

Added by Frédéric Péters 3 months ago. Updated 2 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
09 Jun 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Ça serait idéalement arriver tout seul via le rayonnement (#14176) sauf que ça reste vague, et donc plus ancienne idée encore (#13387), créer des actions explicites sur certaines API. Ici, les notifications.

Pour discuter du bien fondé et de l'utilisation qui en sera faite, #33816.

0001-workflows-add-notification-action-33817.patch View (9.82 KB) Frédéric Péters, 09 Jun 2019 09:20 AM

0001-workflows-add-notification-action-33817.patch View (9.71 KB) Frédéric Péters, 19 Jun 2019 04:15 PM

0001-workflows-add-notification-action-33817.patch View (9.72 KB) Frédéric Péters, 19 Jun 2019 06:02 PM


Related issues

Related to Publik - Development #33816: Avoir une action de WF "Notification" Fermé 09 Jun 2019
Related to w.c.s. - Autre #13387: actions "publik" ? Nouveau 03 Oct 2016

Associated revisions

Revision 82758404 (diff)
Added by Frédéric Péters 2 months ago

workflows: add notification action (#33817)

History

#1 Updated by Frédéric Péters 3 months ago

#2 Updated by Frédéric Péters 3 months ago

#3 Updated by Frédéric Péters 3 months ago

#4 Updated by Frédéric Péters 2 months ago

Avec un fill_admin_form inutile en moins.

#5 Updated by Thomas Noël 2 months ago

Je me pose une question : get_publisher().get_site_option('portal_url', 'variables') ... quid des notifications pour les agents (sur le portail agent ?)

#6 Updated by Frédéric Péters 2 months ago

Oui, et l'autre quid, similaire, c'est les déploiements multi-publik, où on veut plutôt peut-être taper sur le publik de l'interco; je reportais ces questions à un peu plus tard (envie de regarder à nouveau pour exploiter des bouts d'hobo, pour pouvoir chercher les combo dans settings.KNOWN_SERVICES, plutôt qu'empiler/dupliquer ça dans le check_hobo.py).

Pour ces deux cas, en attendant, je laissais le taf à l'action d'appel webservice.

#7 Updated by Thomas Noël 2 months ago

Dans cette attente et pour éviter des incompréhensions, est-ce qu'on pourrait quelque part préciser qu'il s'agit de notification vers le portail usager ? Genre, afficher « Notifications envoyées sur {{portal_url}} » ?

(rien à voir, je me dis qu'il faut penser à faire une api de "notification à un rôle" ou "notification à une liste d'utilisateur" dans combo, parce que là quand il va y avoir plus de 20 personnes dans le rôle, ça peut vite casser... ok avec ça ?)

#8 Updated by Frédéric Péters 2 months ago

Renommer en "Notification à l'usager", et dégager le "by" des paramètres, ok pour ce temps ?

#10 Updated by Frédéric Péters 2 months ago

interdiff,

@@ -28,13 +28,13 @@ from .wscall import WebserviceCallStatusItem

 class SendNotificationWorkflowStatusItem(WebserviceCallStatusItem):
-    description = N_('Notification')
+    description = N_('User Notification')
     key = 'notification'
     category = 'interaction'
     support_substitution_variables = True

     # parameters
-    to = []
+    to = ['_submitter']
     title = None
     body = None
     origin = None
@@ -68,7 +68,7 @@ class SendNotificationWorkflowStatusItem(WebserviceCallStatusItem):
         return self.description

     def get_parameters(self):
-        return ('to', 'title', 'body', 'origin', 'condition')
+        return ('title', 'body', 'origin', 'condition')

     def add_parameters_widgets(self, form, parameters, prefix='', formdef=None):

#11 Updated by Thomas Noël 2 months ago

  • Status changed from Solution proposée to Solution validée

#12 Updated by Frédéric Péters 2 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 82758404c238e39b4c3b68b580a32eb59464dcd9
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun Jun 9 09:10:18 2019 +0200

    workflows: add notification action (#33817)

#13 Updated by Frédéric Péters 2 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF