Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Envoyer en un batch les alertes Joi pour le métier (PIX-15124) #10503

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

dianeCdrPix
Copy link
Contributor

@dianeCdrPix dianeCdrPix commented Nov 6, 2024

🍂 Problème

Quand il y a des erreurs dans les json de Module, transmettre les erreurs en une seule fois, et pas en plusieurs passages.

🌰 Proposition

Utiliser external, async et errors pour que le moduleSchema.validateAsync fonctionne en un batch.

🎃 Remarques

  • .custom() est uniquement synchrone et ne gère pas l'asynchrone : https://joi.dev/api/?v=17.13.3#anycustommethod-description
  • pour valider le html contenu dans certains Element, une validation asynchrone est nécéssaire
  • les tests ici sont pour l'instant KO, car on veut voir le rendu de la console pour les alertes Joi

🪵 Pour tester

  • Lancer les tests API de devComp
  • Constater que toutes les erreurs du json du didacticiel sont remontées dans la console en une fois

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@dianeCdrPix dianeCdrPix changed the title [FEATURE] Envoyer en un batch les alertes Joi pour le métier [FEATURE] Envoyer en un batch les alertes Joi pour le métier (PIX-15124) Nov 8, 2024
@dianeCdrPix dianeCdrPix force-pushed the pix-15124-improve-joi-errors branch 2 times, most recently from 09966ac to 39fd310 Compare November 8, 2024 13:30
@dianeCdrPix dianeCdrPix marked this pull request as ready for review November 8, 2024 13:39
@dianeCdrPix dianeCdrPix requested a review from a team as a code owner November 8, 2024 13:39
@pix-service-auto-merge pix-service-auto-merge merged commit 5916374 into dev Nov 12, 2024
7 of 9 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15124-improve-joi-errors branch November 12, 2024 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants