entrouvert
/
chrono
14
1
Fork 0
Code Issues Pull Requests 4 Releases Wiki Activity

Import CSV, rapporter toutes les erreurs d'un coup (#70523) #130

Merged
vdeniaud merged 2 commits from wip/70523-Import-CSV-rapporter-toutes-les- into main 2023-08-16 09:46:26 +02:00
Conversation 0 Commits 2 Files Changed 2 +158 -128
vdeniaud commented 2023-08-10 11:49:37 +02:00
Owner
Copy Link

Où l'on se rend compte que le patch de #66184 c'est n'importe quoi, plein d'endroits où il y a oubli de substituer ValidationError par ValidationErrorWithOrdinal, et ça passe grâce aux tests qui ne vérifient pas systématiquement le numéro de ligne (qui donc affiche un joli {event_no} dans le texte).

Comme je fais dégager ce ValidationErrorWithOrdinal ça corrige ça, également je complète les tests, également j'uniformise un peu les messages d'erreur entre validation du fichier et validation du modèle.

Où l'on se rend compte que le patch de #66184 c'est n'importe quoi, plein d'endroits où il y a oubli de substituer ValidationError par ValidationErrorWithOrdinal, et ça passe grâce aux tests qui ne vérifient pas systématiquement le numéro de ligne (qui donc affiche un joli `{event_no}` dans le texte). Comme je fais dégager ce ValidationErrorWithOrdinal ça corrige ça, également je complète les tests, également j'uniformise un peu les messages d'erreur entre validation du fichier et validation du modèle.
1691661054.png
33 KiB
1691661054.png
vdeniaud added 2 commits 2023-08-10 11:49:37 +02:00
vdeniaud changed title from WIP: Import CSV, rapporter toutes les erreurs d'un coup (#70523) to Import CSV, rapporter toutes les erreurs d'un coup (#70523) 2023-08-10 12:04:54 +02:00
fpeters approved these changes 2023-08-11 08:31:02 +02:00
vdeniaud merged commit 8127fbff66 into main 2023-08-16 09:46:26 +02:00
vdeniaud deleted branch wip/70523-Import-CSV-rapporter-toutes-les- 2023-08-16 09:46:26 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
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/chrono#130
No description provided.
Delete Branch "wip/70523-Import-CSV-rapporter-toutes-les-"

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?