entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

workflows: reword webservice error recording options (#54931) #1191

Merged
fpeters merged 1 commits from wip/54931-reword-wscall-error-recording-options into main 2024-03-04 13:45:53 +01:00
Conversation 4 Commits 1 Files Changed 2 +15 -6
fpeters commented 2024-02-25 20:20:29 +01:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/54931-reword-wscall-error-recording-options from 9d0a25c464 to f93a1cea36 2024-02-25 20:41:47 +01:00 Compare
fpeters force-pushed wip/54931-reword-wscall-error-recording-options from f93a1cea36 to d47d2f6171 2024-02-26 11:24:28 +01:00 Compare
fpeters reviewed 2024-02-26 11:52:38 +01:00
wcs/wf/wscall.py Outdated
@ -189,2 +188,3 @@
'record_on_errors',
'record_errors',
'record_on_errors',
'notify_on_errors',
fpeters commented 2024-02-26 11:52:01 +01:00
Author
Owner
Copy Link

Modification à l'ordre, on obtient ainsi une progression dans les notifications : (d'abord l'agent, ensuite l'admin fonctionnel, ensuite l'admin système) :

  • Include error details in the log in backoffice
  • Record errors
    • Errors will be available in the logged errors screen.
  • Notify errors by email
    • Error traces will be sent to fred@localhost

soit en traduction rapide :

  • Inclure le détails des erreurs dans le journal en backoffice
  • Enregistrer les erreurs
    • Les erreurs seront disponibles dans l’écran de centralisation des erreurs.
  • Notifier les erreurs par courriel
    • Les traces des erreurs seront envoyées à fred@localhost
Modification à l'ordre, on obtient ainsi une progression dans les notifications : (d'abord l'agent, ensuite l'admin fonctionnel, ensuite l'admin système) : * [x] Include error details in the log in backoffice * [x] Record errors * Errors will be available in the logged errors screen. * [x] Notify errors by email * Error traces will be sent to fred@localhost soit en traduction rapide : * [x] Inclure le détails des erreurs dans le journal en backoffice * [x] Enregistrer les erreurs * Les erreurs seront disponibles dans l’écran de centralisation des erreurs. * [x] Notifier les erreurs par courriel * Les traces des erreurs seront envoyées à fred@localhost
wcs/wf/wscall.py Outdated
@ -379,3 +379,3 @@
)
if 'notify_on_errors' in parameters:
if 'notify_on_errors' in parameters and get_publisher().logger.error_email:
fpeters commented 2024-02-26 11:52:24 +01:00
Author
Owner
Copy Link

Avec cette dernière option présentée uniquement quand une adresse a été définie.

Avec cette dernière option présentée uniquement quand une adresse a été définie.
fpeters changed title from WIP: workflows: reword webservice error recording options (#54931) to workflows: reword webservice error recording options (#54931) 2024-02-26 11:52:43 +01:00
ecazenave requested changes 2024-02-26 14:29:28 +01:00
Dismissed
ecazenave left a comment
Owner
Copy Link
  • Inclure le détails des erreurs dans le journal en backoffice
  • Enregistrer les erreurs
    • Les erreurs seront disponibles dans l’écran de centralisation des erreurs.

La distinction entre ceux deux-là me semble encore bien cryptique, plutôt que d'adapter l'ordre et les libellés je serais pour fusionner ces deux options (je dis ça sans savoir s'il y a des cas d'usages à cette finesse).

> * [x] Inclure le détails des erreurs dans le journal en backoffice > * [x] Enregistrer les erreurs > * Les erreurs seront disponibles dans l’écran de centralisation des erreurs. La distinction entre ceux deux-là me semble encore bien cryptique, plutôt que d'adapter l'ordre et les libellés je serais pour fusionner ces deux options (je dis ça sans savoir s'il y a des cas d'usages à cette finesse).
fpeters commented 2024-02-26 14:34:44 +01:00
Author
Owner
Copy Link
  • Inclure le détails des erreurs dans le journal en backoffice
  • Enregistrer les erreurs
    • Les erreurs seront disponibles dans l’écran de centralisation des erreurs.

La distinction entre ceux deux-là me semble encore bien cryptique, plutôt que d'adapter l'ordre et les libellés je serais pour fusionner ces deux options (je dis ça sans savoir s'il y a des cas d'usages à cette finesse).

Si on fusionne on impose l'affichage de ces erreurs à des agents fonctionnels qui ne capteront rien.

À rester dans de la reformulation, c'est "Enregistrer les erreurs" qui est trop large, je pense, il faudrait quelque chose dès le libellé qui pointe que c'est dans l'écran central des erreurs que ça enregistre ? "Enregistrer le détail technique de l’erreur pour l’équipe d'administration" ?

> > * [x] Inclure le détails des erreurs dans le journal en backoffice > > * [x] Enregistrer les erreurs > > * Les erreurs seront disponibles dans l’écran de centralisation des erreurs. > > La distinction entre ceux deux-là me semble encore bien cryptique, plutôt que d'adapter l'ordre et les libellés je serais pour fusionner ces deux options (je dis ça sans savoir s'il y a des cas d'usages à cette finesse). Si on fusionne on impose l'affichage de ces erreurs à des agents fonctionnels qui ne capteront rien. À rester dans de la reformulation, c'est "Enregistrer les erreurs" qui est trop large, je pense, il faudrait quelque chose dès le libellé qui pointe que c'est dans l'écran central des erreurs que ça enregistre ? "Enregistrer le détail technique de l’erreur pour l’équipe d'administration" ?
fpeters dismissed ecazenave’s review 2024-02-28 10:35:27 +01:00
Reason:

cf questions que je pose.

ecazenave commented 2024-03-04 12:36:58 +01:00
Owner
Copy Link

Peut-être être encore plus direct ?

  • afficher les erreurs aux agents
  • afficher les erreurs aux administrateurs
Peut-être être encore plus direct ? * afficher les erreurs aux agents * afficher les erreurs aux administrateurs
fpeters commented 2024-03-04 13:09:01 +01:00
Author
Owner
Copy Link

Ok préciser le public cible, et pour être complet combiner avec la localisation, quelque chose de cet ordre :

Enregistrer les erreurs dans l’historique de la demande, pour visualisation par les agents
Enregistrer les erreurs dans l’écran de centralisation des erreurs, pour traitement par l’équipe d’administration

?

Ok préciser le public cible, et pour être complet combiner avec la localisation, quelque chose de cet ordre : > Enregistrer les erreurs dans l’historique de la demande, pour visualisation par les agents > Enregistrer les erreurs dans l’écran de centralisation des erreurs, pour traitement par l’équipe d’administration ?
ecazenave commented 2024-03-04 13:23:13 +01:00
Owner
Copy Link

Vendu.

Vendu.
ecazenave approved these changes 2024-03-04 13:23:21 +01:00
fpeters force-pushed wip/54931-reword-wscall-error-recording-options from d47d2f6171 to 8de50e6e5d 2024-03-04 13:33:35 +01:00 Compare
fpeters merged commit d10392f0fe into main 2024-03-04 13:45:53 +01:00
fpeters deleted branch wip/54931-reword-wscall-error-recording-options 2024-03-04 13:45:53 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
ecazenave
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#1191
No description provided.
Delete Branch "wip/54931-reword-wscall-error-recording-options"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?