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

[TECH] Améliorer les notifications d'erreur sur la création de profil cible #4411

Merged

Conversation

nlepage
Copy link
Member

@nlepage nlepage commented May 9, 2022

🦄 Problème

Lorsqu'on saisit un identifiant d'organisation de référence qui n'existe pas pour créer un profil cible, on reçoit une notification d'erreur générique "Une erreur est survenue".

🤖 Solution

Afficher une notification d'erreur plus spécifique "Organisation non trouvée".

🌈 Remarques

N/A

💯 Pour tester

La notification d'erreur doit être "Organisation non trouvée"

⚠️ Si l'id saisi est trop long, c'est "Erreur lors de la création du profil cible" qui s'affiche.

@nlepage nlepage added 👀 Tech Review Needed team-contenu PR relatives à l'équipe Contenu labels May 9, 2022
@nlepage nlepage self-assigned this May 9, 2022
@pix-service
Copy link
Contributor

Copy link
Contributor

@Jeremiejade Jeremiejade left a comment

Choose a reason for hiding this comment

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

¡ LGTM !

@nlepage nlepage changed the title [TECH] Améliorer [TECH] Améliorer les notifications d'erreur sur la création de profil cible May 9, 2022
@nlepage nlepage added the cross-team Toutes les équipes de dev label May 10, 2022
@pix-service-auto-merge pix-service-auto-merge force-pushed the tech-createtargetprofile-error-notification branch from 00a61d0 to 8196552 Compare May 17, 2022 12:51
@pix-service-auto-merge pix-service-auto-merge deleted the tech-createtargetprofile-error-notification branch May 17, 2022 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev 🚀 Ready to Merge team-contenu PR relatives à l'équipe Contenu Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants