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

ux(fix): make custom errors modal title dynamic #2464

Merged
merged 3 commits into from
Nov 14, 2024

Conversation

Holist
Copy link
Collaborator

@Holist Holist commented Nov 13, 2024

Le titre de la modale des erreurs customs était en dur et donc pas vraiment custom... 😄

@Holist Holist self-assigned this Nov 13, 2024
Copy link
Collaborator

@aminedhobb aminedhobb left a comment

Choose a reason for hiding this comment

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

Merci @Holist pour le fix 🙏 !
Ce n'est pas le scope de cette PR mais je me rends compte que je ne vois pas la différence entre utiliser l'error_modal et la custom_errors_modal.
Est-ce qu'on ne devrait pas toujours utiliser la custom_errors_modal vu qu'elle prend en compte tous types d'erreurs (custom et non custom) ?

@Holist
Copy link
Collaborator Author

Holist commented Nov 14, 2024

Merci @Holist pour le fix 🙏 ! Ce n'est pas le scope de cette PR mais je me rends compte que je ne vois pas la différence entre utiliser l'error_modal et la custom_errors_modal. Est-ce qu'on ne devrait pas toujours utiliser la custom_errors_modal vu qu'elle prend en compte tous types d'erreurs (custom et non custom) ?

Oui tu as raison.
Il faudra faire la même chose pour les messages flash aussi éventuellement.
Je créé des tickets tech en ce sens.

@Holist
Copy link
Collaborator Author

Holist commented Nov 14, 2024

Done here #2476

@Holist Holist merged commit 959db27 into staging Nov 14, 2024
7 checks passed
@Holist Holist deleted the ux_fix_custom_errors_modal_title branch November 14, 2024 10:41
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.

2 participants