AstrolabeCAE/SiteWebAstrolabe
AstrolabeCAE/SiteWebAstrolabe
2
2
Fork 5
Code Issues 2 Pull Requests Packages Projects Releases 3 Activity

feat/send-mail-to-member #81

Merged
florent.lesaout merged 10 commits from feat/send-mail-to-member into master 2024-03-20 14:04:39 +01:00
Conversation 7 Commits 10 Files Changed 15 +224 -121
jeremy.dufraisse commented 2024-03-18 15:27:05 +01:00
Collaborator
Copy Link

Je propose cette branche pour pouvoir avoir un formulaire pour écrire aux entrepreneurs et entrepreneures.

Je laisse encore en brouillon car j'ai deux points qui ne me plaisent pas encore:

  • je trouve qu'il faudrait adapter le design (css) du formulaire pour écrire à un membre
  • il faut tester que toute la boucle fonctionne bien
  • la génération de l'adresse e-ma se fait à partir du fileSlug de la fiche d'un membre en remplaçant - par ..
    • il faudrait rendre ceci un peu plus robuste pour les adresses e-mail avec tiret (usage du emailSlug)
Je propose cette branche pour pouvoir avoir un formulaire pour écrire aux entrepreneurs et entrepreneures. Je laisse encore en brouillon car j'ai deux points qui ne me plaisent pas encore: - [x]je trouve qu'il faudrait adapter le design (css) du formulaire pour écrire à un membre - [x] il faut tester que toute la boucle fonctionne bien - [x] la génération de l'adresse e-ma se fait à partir du `fileSlug` de la fiche d'un membre en remplaçant `-` par `.`. - [x] il faudrait rendre ceci un peu plus robuste pour les adresses e-mail avec tiret (usage du `emailSlug`)
florent.lesaout requested review from kevin.avrillon 2024-03-18 21:28:43 +01:00
florent.lesaout requested review from florent.lesaout 2024-03-18 21:28:55 +01:00
florent.lesaout commented 2024-03-19 11:53:07 +01:00
Owner
Copy Link

Je viens de tester et quand je clique sur envoyer, j'ai ça, mais est-ce peut-être normal car ce n'est pas encore prêt ?
image

Je viens de tester et quand je clique sur envoyer, j'ai ça, mais est-ce peut-être normal car ce n'est pas encore prêt ? ![image](/attachments/ab1af3fb-f887-41e1-a72f-b931ae1d065b)
image.png
152 KiB
jeremy.dufraisse commented 2024-03-19 12:16:45 +01:00
Author
Collaborator
Copy Link

Salut,
alors en local, tu n'as peut-être pas activé ton serveur php (eleventy fonctionne avec nodejs et n'active pas le serveur php avec pour serveur httpd local - NGINX ou Apache) c'est pour ça que j'ai dit que je n'avais pas encore testé.

En plus, il faut faire attention à ce que l'instance PHP configure la bonne variable environnement pour l'adresse e-mail de base

Salut, alors en local, tu n'as peut-être pas activé ton serveur `php` (`eleventy` fonctionne avec `nodejs` et n'active pas le serveur `php` avec pour serveur `httpd` local - `NGINX` ou `Apache`) c'est pour ça que j'ai dit que je n'avais pas encore testé. En plus, il faut faire attention à ce que l'instance PHP configure la bonne variable environnement pour l'adresse e-mail de base
florent.lesaout commented 2024-03-19 14:20:31 +01:00
Owner
Copy Link

Ah oui je vois donc en gros c'est normal 👍

Je ne suis pas expert mais je voulais voir ce que ça donne

Ah oui je vois donc en gros c'est normal 👍 Je ne suis pas expert mais je voulais voir ce que ça donne
jeremy.dufraisse force-pushed feat/send-mail-to-member from ea1ceafe56 to 9690867817 2024-03-20 13:10:37 +01:00 Compare
jeremy.dufraisse changed title from WIP: feat/send-mail-to-member to feat/send-mail-to-member 2024-03-20 13:11:27 +01:00
jeremy.dufraisse commented 2024-03-20 13:12:27 +01:00
Author
Collaborator
Copy Link

@florent.lesaout cette branche est bonne pour fusion de mon côté (j'ai testé, le côté PHP, ça devrait être bon, le design me convient mieux.)

@florent.lesaout cette branche est bonne pour fusion de mon côté (j'ai testé, le côté PHP, ça devrait être bon, le design me convient mieux.)
jeremy.dufraisse commented 2024-03-20 13:16:12 +01:00
Author
Collaborator
Copy Link
No description provided.
Capture d’écran 2024-03-20 à 13.14.57.png
68 KiB
Capture d’écran 2024-03-20 à 13.14.57.png
florent.lesaout commented 2024-03-20 13:55:06 +01:00
Owner
Copy Link

Salut,

Si le test est fonctionnel de ton côté, et que je ne peux pas localement tester je te fais confiance.
De toute façon nous pouvons déjà déployer cela et aller dans une démarche d'amélioration continue si cela ne nous donne pas entièrement satisfaction.

Ce sera déjà une très bonne amélioration et très utile pour le dev eco des membres 👍

Merci pour le taf

Salut, Si le test est fonctionnel de ton côté, et que je ne peux pas localement tester je te fais confiance. De toute façon nous pouvons déjà déployer cela et aller dans une démarche d'amélioration continue si cela ne nous donne pas entièrement satisfaction. Ce sera déjà une très bonne amélioration et très utile pour le dev eco des membres 👍 Merci pour le taf
florent.lesaout merged commit 9fd1712328 into master 2024-03-20 14:04:39 +01:00
florent.lesaout deleted branch feat/send-mail-to-member 2024-03-20 14:04:46 +01:00
florent.lesaout commented 2024-03-20 14:08:28 +01:00
Owner
Copy Link

Déployé et testé avec succès :
image

Déployé et testé avec succès : ![image](/attachments/f38acc4c-8872-43eb-8df4-3ca3aba5f0a9)
image.png
89 KiB
Sign in to join this conversation.
Reviewers
No reviewers
kevin.avrillon
florent.lesaout
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: AstrolabeCAE/SiteWebAstrolabe#81
No description provided.
Delete Branch "feat/send-mail-to-member"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?