Skip to content
This repository has been archived by the owner on Feb 13, 2023. It is now read-only.

Release #29

Closed
121593 opened this issue Apr 5, 2020 · 12 comments · Fixed by #31
Closed

Release #29

121593 opened this issue Apr 5, 2020 · 12 comments · Fixed by #31

Comments

@121593
Copy link
Contributor

121593 commented Apr 5, 2020

Il y a eu pas mal de changements faits dernièrement.
Il serait peut-être l'heure de clôturer les PR restantes, tester l'ensemble et envisager une release
Qu'en pensez-vous?

@nicolasdb
Copy link
Member

Salut,
Oui. je donne ici une réponse générale pour #17 et #22
Techniquement, je pense que c'est ok. Mais pour la validation du contenu, aujourd'hui, je suis encore tout seul à y réfléchir donc, ça n'avance pas vite.

Est-ce qu'on pourrait se faire une visio Loic?
ça m'aiderai de refaire le point sur les nouveautés et les possibilités.

Avec la pandémie et le nouveau CA et la nouvelle méthode d'inscription et aussi j'ai décidé de laisser tomber les services aux clients. Je veux simplifier la gestion de l'asbl et recentrer sur le minimum viable pour tenir le lieu ouvert pour les membres
En gros, y a du gros taf sur l'organisation des pages et le contenu.

Est-ce qu'on peut faire une release beta pour valider que tout fonctionne bien?
Je ferai une modif de la page de garde pour annoncer les changements de base et communiquer sur la situation actuelle, les activités de productions de visières et qu'on reste actif.

Et se laisser un peu de temps pour monter le contenu?

@121593
Copy link
Contributor Author

121593 commented Apr 14, 2020

Salut
Pour la #17 Il faut juste que tu valides les 2 lignes de texte dans profil.ejs, voir ici

En gros, y a du gros taf sur l'organisation des pages et le contenu.

L'objectif de ce ticket est de faire une release rapide de ce qui a d'ores et déjà été fait. On pourrait intégrer #22 et te laisser travailler dessus par la suite

Est-ce qu'on peut faire une release beta pour valider que tout fonctionne bien?

À ma connaissance il n'y pas d'environnement de test en ligne. Mais une fois les PR intégrées chacun pourra valider les changements en local.

@nicolasdb
Copy link
Member

non, c'est ok, l'installation docker et tout ça, c'est ok pour le test du contenu.

@nicolasdb
Copy link
Member

Dites, la release c'est juste sur le master git ou en ligne?

je teste un peu zapier pour trier les mails et faire des issues mais même avec mon adresse gmail perso, j'arrive dans les spam d'openfab.
C'est donc prévu que tout ça aille mieux suite à ce que tu as fait, right? @121593

image

@satblip
Copy link
Member

satblip commented Apr 21, 2020

Pour moi, un des gros fix des spams est plutôt le contenu du mail qui doit être plus adapté.

un test peut être réalisé avec https://www.mail-tester.com/

@satblip
Copy link
Member

satblip commented Apr 21, 2020

C'est marrant, un grand pourcentage a l'air de passer, ils arrivent tous dans les spams?
image

@satblip
Copy link
Member

satblip commented Apr 21, 2020

Je viens de check, mais point de vue config serveur, on est bon :
image
(result : https://www.mail-tester.com/test-foru78tql)

Le problème vient d'office qu'on fait le from email depuis le user, et on peut plus faire ça : https://github.com/openfab-lab/app-membres/blob/master/src/controllers/contact.js#L39
Il vaudrait mieux faire un from depuis config.email.from et setup un replyTo dans le payload de l'email. Cela devrait résoudre le problème

@nicolasdb
Copy link
Member

ben, je n'en ai testé que 2, un avec une adresse bullshit et un avec mon adresse perso.
Après, c'est vrai que j'ai fait le troll client dans le message mais ça m'étonnerai quand même que le filtre comprennent à ce point le contenu.
msg1, parlait d'une demande pour un projet en acier pur et que j'avais un budget de 3€
msg2, parlait d'une demande d'info par un prince wallon, qui voulait me léger 100€ pour faire plaisir.
:)

Je collecte tjs autant de spam, ce matin 2 nouveaux.
celui ci ce matin. pas un spam.
image

L'autre est bien un spam.
image

Mais le captcha n'est tjs pas en ligne, donc, est-ce que la nouvelle release est active?
@satblip, Comment ça marche les étapes jusqu'au serveur?
ça serait intéressant d'avoir ça sur le wiki, à titre informatif.

@satblip
Copy link
Member

satblip commented Apr 22, 2020

Pour moi, on doit changer la from address, comme j'ai pointé dans mon précédent message. Le captcha va empècher les bots, pas que nos mails tombent dans les spams. J'attends aussi que cette PR soit closed : #30 et puis je deploy ;)

@satblip
Copy link
Member

satblip commented Apr 29, 2020

@121593 j'ai deployed (migration incluse), mais la member page is broken now : https://openfab.be/membres/qkSOdDnV5c

@satblip
Copy link
Member

satblip commented Apr 30, 2020

@121593 @nicolasdb Voila released deployed et fonctionnelle : https://openfab.be/membres/qkSOdDnV5c

@nicolasdb
Copy link
Member

Merci les copains.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants