Skip to content

Conversation

@HubM
Copy link
Collaborator

@HubM HubM commented Dec 23, 2025

  • ✨ Script pour récupérer les fichiers détails des candidature
  • ✅ Ajout d'une nouvelle projection détail-candidature en cas d'import de nouveau candidat

@HubM HubM force-pushed the pot-2299-migrer-les-details branch 2 times, most recently from f5f8a34 to c75aa99 Compare December 23, 2025 10:16
@HubM HubM marked this pull request as ready for review January 8, 2026 10:47
Comment on lines 1689 to 1693
'candidature.consulterDétail',
'candidature.importer',
'candidature.importerDétail',
'candidature.corriger',
'candidature.corrigerDétail',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

je ne pense pas que ce soit une fonctionalité en plus, tu peux inclure la commande dans consulter/importer/corriger non ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

j'ai surtout oublié de virer my bad

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

est-ce pour correspondre mieux à l'existant on ne ferait pas un RawType ? pour avoir `DétailCandidature.RawType

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

et d'ailleurs, tu pourrais aussi ajouter un ValueType, et du coup tu as l'endroit où mettre le helper dont on parlait hier...

"scripts": {
"build": "tsc"
"build": "tsc",
"test": "tsx --test src/**/*.spec.ts"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pas utilisé

if (payload.détailsValue && Object.keys(payload.détailsValue).length > 0) {
const buf = Buffer.from(JSON.stringify(payload.détailsValue));
const blob = new Blob([buf]);
await mediator.send<EnregistrerDocumentProjetCommand>({
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on garde la fichier ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai rien fais là dessus pour le moment parce que j'étais pas sur, faudrait demander au métier ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on peut garder pour le moment, on supprimera une fois la solution éprouvée ?
le métier ne saura pas de quoi tu parles ^^ c'est un sujet purement tech.

@HubM HubM force-pushed the pot-2299-migrer-les-details branch from cf5e2c8 to 8f983fa Compare January 8, 2026 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants