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

api: skip elements with invalid slugs in export-import API (#83124) #814

Merged
fpeters merged 1 commits from wip/83124-export-import-api-invalid-slugs into main 2023-11-14 10:11:33 +01:00
Conversation 0 Commits 1 Files Changed 2 +40 -11
fpeters commented 2023-11-05 08:49:05 +01:00
Owner
Copy Link
No description provided.
fpeters added 1 commit 2023-11-05 08:49:06 +01:00
fpeters reviewed 2023-11-05 09:35:54 +01:00
wcs/api_export_import.py
@ -195,2 +198,3 @@
raise Http404()
return JsonResponse({'data': [export_object_ref(request, x) for x in klass.select()]})
object_refs = [export_object_ref(request, x) for x in klass.select()]
return JsonResponse({'data': [x for x in object_refs if x]})
fpeters commented 2023-11-05 09:35:54 +01:00
Author
Owner
Copy Link

Je ne suis pas totalement fan de juste zappper ces éléments mais je n'ai pas d'idée propre pour la remontée de l'info comme quoi il y a un bout en erreur dans hobo.

Je ne suis pas totalement fan de juste zappper ces éléments mais je n'ai pas d'idée propre pour la remontée de l'info comme quoi il y a un bout en erreur dans hobo.
lguerin commented 2023-11-06 10:21:31 +01:00
Owner
Copy Link

Peut-être sur le NoReverseMatch, poser un log.error/warning/peu importe, ou une logged error ?
Pour qu'on puisse corriger les slugs malformés, éventuellement

Peut-être sur le NoReverseMatch, poser un log.error/warning/peu importe, ou une logged error ? Pour qu'on puisse corriger les slugs malformés, éventuellement
fpeters changed title from WIP: api: skip elements with invalid slugs in export-import API (#83124) to api: skip elements with invalid slugs in export-import API (#83124) 2023-11-05 09:36:01 +01:00
lguerin approved these changes 2023-11-14 10:09:25 +01:00
fpeters merged commit d470d5068b into main 2023-11-14 10:11:33 +01:00
fpeters deleted branch wip/83124-export-import-api-invalid-slugs 2023-11-14 10:11:33 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
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#814
No description provided.
Delete Branch "wip/83124-export-import-api-invalid-slugs"

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?