🚸 Vérifier la présence des colonnes dans le csv #3664
Conversation
HubM
changed the title
…ur prévenir des erreurs dans les colonnes existantes
HubM
force-pushed
the
pot-2093-verifier-la-presence-des-colonnes-dans-le-csv
branch
from
74691a3 to
d87abc5
Compare
|
@copilot make suggestions |
| candidatureCsvSchema, | ||
| { encoding: 'win1252', delimiter: ';' }, | ||
| ); | ||
| const { parsedData, rawData } = await parseCsv({ |
There was a problem hiding this comment.
Je me demande s'il ne faudrait pas vérifier les colonnes aussi lors de la correction par lot. Parce que c'est ce qui est arrivé l'autre jour, Tiphany essayait d'importer un fichier de correction avec une mauvaise colonne.
| const { parsedData, rawData } = await parseCsv({ | ||
| fileStream: fichierImportCandidature.content, | ||
| lineSchema: candidatureCsvSchema, | ||
| columnsToVerify: Object.values(candidatureCsvHeadersMapping), |
There was a problem hiding this comment.
Il faudrait peut être une fonction intermédiaire pour filtrer les colonnes selon l'AO. Sinon on va avoir des erreurs pour les AOs qui n'ont pas les colonnes spécifiques à Petit PV. On peut se baser sur l'AO entity > Champs supplémentaires.
D'ailleurs ces colonnes ne devraient jamais être dans un csv de candidats. Sauf si on veut pouvoir corriger par lot. Sinon ça sert juste pour nos tests.
There was a problem hiding this comment.
Aussi toutes les colonnes relatives à la typologie ne sont pas demandées pour tous les AOs. Par exemple si les colonnes sont manquantes pour une période de PPE2 - Eolien, je ne suis pas sûre qu'il faille bloquer l'import. Le rapprochement entre ces colonnes et les AOs n'est pas encore matérialisé dans les l'entity AO il me semble, donc à discuter pour voir comment on met ça en place !
| fileStream: ReadableStream; | ||
| lineSchema: TSchema; | ||
| parseOptions?: Partial<ParseOptions>; | ||
| columnsToVerify?: ReadonlyArray<string>; |
There was a problem hiding this comment.
| columnsToVerify?: ReadonlyArray<string>; | |
| requiredColumns?: ReadonlyArray<string>; |
No description provided.