entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 15 Releases Activity

toulouse-maelis: log soap on get-recurrent-week (#84975) #430

Merged
nroche merged 1 commits from wip/84975-parsifal-log-soap-on-recurrent-week into main 2023-12-21 11:11:44 +01:00
Conversation 1 Commits 1 Files Changed 3 +532 -11
nroche commented 2023-12-20 14:05:23 +01:00
Owner
Copy Link

Expliciter d'avantage les erreurs de paramétrage Maélis sur la semaine types des réservations.

Expliciter d'avantage les erreurs de paramétrage Maélis sur la semaine types des réservations.
nroche added 1 commit 2023-12-20 14:05:23 +01:00
gitea/passerelle/pipeline/head There was a failure building this commit Details
17147347a7 toulouse-maelis: log soap on get-recurrent-week (#84975)
nroche force-pushed wip/84975-parsifal-log-soap-on-recurrent-week from 17147347a7 to 2152cf6b38 2023-12-20 16:27:20 +01:00 Compare
bdauvergne reviewed 2023-12-20 16:34:58 +01:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -2505,12 +2505,20 @@ class ToulouseMaelis(BaseResource, HTTPResource):
response = self.call('Activity', 'getPersonScheduleList', **payload)
date_min_prev = None
warning_msg = 'No week calendar for activity %s on %s' % (
bdauvergne commented 2023-12-20 16:34:58 +01:00
Owner
Copy Link

Dans ce cas pas d'APIError et pas de log (ni erreur ni warning), c'est bien ce qui est voulu ?

Dans ce cas pas d'APIError et pas de log (ni erreur ni warning), c'est bien ce qui est voulu ?
nroche commented 2023-12-20 16:39:39 +01:00
Author
Owner
Copy Link

Non tu as raison, j'ai tapé trop large.
Mon idée est de ne pas avoir de warning si le endpoint est appelé sans passer d'identifiant d'activité.
(je veux dire avec activity_id à 'None' pour retirer ces nombreux warnings)

Non tu as raison, j'ai tapé trop large. Mon idée est de ne pas avoir de warning si le endpoint est appelé sans passer d'identifiant d'activité. (je veux dire avec activity_id à 'None' pour retirer ces nombreux warnings)
bdauvergne marked this conversation as resolved
nroche force-pushed wip/84975-parsifal-log-soap-on-recurrent-week from 2152cf6b38 to 24ec27ca21 2023-12-20 16:59:07 +01:00 Compare
bdauvergne approved these changes 2023-12-20 17:20:38 +01:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -2544,1 +2545,3 @@
}
if date_min_prev:
date_min_prev = date_min_prev.strftime(utils.json_date_format)
if warning_msg and "'None'" not in warning_msg:
bdauvergne commented 2023-12-20 17:20:34 +01:00
Owner
Copy Link

Ok mais il faudra un commentaire ici, sinon c'est cryptique.

Ok mais il faudra un commentaire ici, sinon c'est cryptique.
nroche commented 2023-12-21 11:11:16 +01:00
Author
Owner
Copy Link

(fait: j'ai ajouté un commentaire)

(fait: j'ai ajouté un commentaire)
nroche force-pushed wip/84975-parsifal-log-soap-on-recurrent-week from 24ec27ca21 to 0010095146 2023-12-21 09:53:38 +01:00 Compare
nroche merged commit 0010095146 into main 2023-12-21 11:11:44 +01:00
nroche deleted branch wip/84975-parsifal-log-soap-on-recurrent-week 2023-12-21 11:11:44 +01:00
bdauvergne commented 2023-12-21 11:30:17 +01:00
Owner
Copy Link
            # do not raise on 'None' activity

Oui c'est évident ça :) Le commentaire demandé c'était d'expliquer la signification de ce None pas de paraphraser le code.

``` # do not raise on 'None' activity ``` Oui c'est évident ça :) Le commentaire demandé c'était d'expliquer la signification de ce None pas de paraphraser le code.
Sign in to join this conversation.
Reviewers
No reviewers
bdauvergne
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/passerelle#430
No description provided.
Delete Branch "wip/84975-parsifal-log-soap-on-recurrent-week"

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?