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

Fix: message de confirmation d'envoi du formulaire de contact #1574

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

@oumeimaelisbihani oumeimaelisbihani marked this pull request as ready for review January 29, 2025 10:10
Copy link
Collaborator

@ttdm ttdm left a comment

Choose a reason for hiding this comment

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

Je pense que j'ai trouvé un comportement non souahité :
image

Le message sans conseiller entreprise pour un projet, envoyé à CE.

Petite remarque sur le process de review, ce serait cool si tu pouvais ajouter des liens directs qui permettent de voir le/les résultats de la PR.
Tu mets un lien vers chaque cas : projet / cutsomprojet / programme pris en charge par CE / projet autonome (non pris en charge par CE )

Edit: ah ben voila, ça évitera ce genre d'erreur de review; j'ai pris le numéro d'une autre de tes PR x) je refait me path fonctionnel !

Copy link
Collaborator

@ttdm ttdm left a comment

Choose a reason for hiding this comment

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

DU coup je viens de faire une path au bon endroit et les custom projet, qui sont envoyé à CE ne sont pas associés au bon message
Petite remarque :
Le logo CE est un poil petit; on voit qu'il y a un sous titre mais il est illisible ce qui est pas très agréable.

@oumeimaelisbihani oumeimaelisbihani requested review from ttdm and removed request for dolemoine January 30, 2025 10:53
Copy link
Collaborator

@ttdm ttdm left a comment

Choose a reason for hiding this comment

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

niquel !
Et merci pour les liens de validations !

@ttdm ttdm requested a review from yvalentin January 30, 2025 11:12
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.

Modifier le message de confirmation d'envoi du formulaire de contact
3 participants