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

toulouse-maelis: always return err=0 on dlnuf endpoint (#86503) #459

Merged
nroche merged 1 commits from wip/86503-parsifal-no-api-error-on-dlnuf-endpoint into main 2024-02-02 16:58:30 +01:00
Conversation 0 Commits 1 Files Changed 2 +4 -6
nroche commented 2024-02-02 16:27:50 +01:00
Owner
Copy Link

Ne pas lever l'APIError, pour ne pas générer de fausses erreurs dans w.c.s.

Ne pas lever l'APIError, pour ne pas générer de fausses erreurs dans w.c.s.
nroche added 1 commit 2024-02-02 16:27:51 +01:00
tnoel requested changes 2024-02-02 16:35:13 +01:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -2348,3 +2348,1 @@
if response is not True:
raise APIError('document is not valid')
return {'data': 'ok'}
return {'data': 'ok' if response is True else 'KO'}
tnoel commented 2024-02-02 16:34:34 +01:00
Owner
Copy Link

Réponds ok / ko, OK / KO, mais ok/KO tu vas perdre du monde :)

Réponds ok / ko, OK / KO, mais ok/KO tu vas perdre du monde :)
nroche force-pushed wip/86503-parsifal-no-api-error-on-dlnuf-endpoint from 3ec9d5e048 to 9a158a66d1 2024-02-02 16:37:37 +01:00 Compare
tnoel approved these changes 2024-02-02 16:38:24 +01:00
tnoel left a comment
Owner
Copy Link

ok ! :)

ok ! :)
nroche merged commit 9a158a66d1 into main 2024-02-02 16:58:30 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
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#459
No description provided.
Delete Branch "wip/86503-parsifal-no-api-error-on-dlnuf-endpoint"

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?