misc: do not include file contents in json export if file is not set (#89876) #1435

Merged
fpeters merged 1 commits from wip/89876-AttachmentEvolutionPart-filename into main 2024-04-22 17:47:58 +02:00
Owner

C'est lia aux tests de workflow, ici j'assure juste que ça ne crashe pas mais je laisse le dernier mot à Valentin, il y a peut-être mieux à faire.

C'est lia aux tests de workflow, ici j'assure juste que ça ne crashe pas mais je laisse le dernier mot à Valentin, il y a peut-être mieux à faire.
fpeters added 1 commit 2024-04-22 16:31:19 +02:00
fpeters requested review from vdeniaud 2024-04-22 16:31:26 +02:00
vdeniaud approved these changes 2024-04-22 17:42:23 +02:00
vdeniaud left a comment
Owner

Ça crashe dans un chemin de code très secondaire (export de la demande dans l'état où elle se trouve après l'exécution des tests pour pouvoir afficher un inspecteur) et en plus export des evolutions qu'on affiche pas, donc go débloquer avec ce patch minimal

Hors pression du bug je vais quand même chercher à comprendre si on ne pourrait pas retrouver le filename ici

Ça crashe dans un chemin de code très secondaire (export de la demande dans l'état où elle se trouve après l'exécution des tests pour pouvoir afficher un inspecteur) et en plus export des evolutions qu'on affiche pas, donc go débloquer avec ce patch minimal Hors pression du bug je vais quand même chercher à comprendre si on ne pourrait pas retrouver le filename ici
fpeters merged commit 0fcf440e0a into main 2024-04-22 17:47:58 +02:00
fpeters deleted branch wip/89876-AttachmentEvolutionPart-filename 2024-04-22 17:47:58 +02:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
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#1435
No description provided.