search

openfab-lab / app-membres

Une application pour la gestion de nos membres / An app to manage our fablab's communty
1 stars 2 forks source link

Release #29

Closed 121593 closed 4 years ago

121593 commented 4 years ago

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 commented 4 years ago

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 commented 4 years ago

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 commented 4 years ago

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

nicolasdb commented 4 years ago

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 commented 4 years ago

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 commented 4 years ago

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

satblip commented 4 years ago

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 commented 4 years ago

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 commented 4 years ago

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 : https://github.com/openfab-lab/app-membres/pull/30 et puis je deploy ;)

satblip commented 4 years ago

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

satblip commented 4 years ago

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

nicolasdb commented 4 years ago

Merci les copains.