Vuejs

[linsolas]

PR faisant suite à l'issue https://github.com/brownbaglunch/BrownBagLunch/issues/341 Quelques points à voir / corriger :

• [x] éviter d'avoir un affichage avec des placeholders en place tant que les données ne sont pas récupérées.
• [x] mettre une image par défaut au bagger sans image / avec une image qui ne charge pas (-> peut être un travail à faire sur le JSON des baggers ?)
• [ ] gestion des baggers multi-villes (le header, par ex., ne prend en compte que la première ville déclarée).

[dadoonet]

@linsolas J'ai créé un repo https://github.com/brownbaglunch/staging. Reste plus qu'à configurer correctement le DNS (ce que je n'ai pas réussi à faire - Gandi me dit que je n'ai pas les droits).

L'idée est de pouvoir faire tout ce qu'on veut sur http://staging.brownbaglunch.fr/ et si tout est ok, faire un merge du gh-pages correct vers gh-pages-origin de staging vers origin (repo brownbaglunch).

Ca nous permettrait de faire des itérations directement visualisables en ligne. Ca rend peut-être le process trop compliqué néanmoins... A voir.

Du coup, pourrais tu envoyer ta PR dans staging? Voir faire un merge dans staging/gh-pages ?

LMK ce que tu en penses. A mercredi ! :D

[linsolas]

Sur le principe c'est clairement plus propre. Maintenant, le site n'a pas eu de grosse mise a jour depuis au moins 2 ans. Pas sûr que ça soit vraiment le bon candidat pour une telle architecture, non ?

[linsolas]

🆙

[dadoonet]

@linsolas Ca t'ennuierait d'essayer manuellement de commiter ta PR sur https://github.com/brownbaglunch/staging ? Le site staging fonctionne visiblement... (ou pas)

[linsolas]

Hello @dadoonet

Je peux le faire, mais quel serait le processus au final ? On a 2 repo différent (l'original et le staging), du coup si je fais une modif sur le staging, comment on la répercute sur l'original ? Ne serait-ce pas plus simple si le staging était une branche de ce repo ?

[dadoonet]

Ça serait l'idéal si GitHub savait le gérer ! Mais pas de multiples pages depuis différentes branches AFAIK.

[linsolas]

ah oui c'est vrai que le site est livré par Github Pages... Mais quid du passage staging -> prod ?

[dadoonet]

Il suffit de faire (pseudo code)

git remote add staging git:...

Puis faire du

git pull staging git merge staging/gh-pages git push origin

(Dans Le genre)

[linsolas]

Yo, 🆙

J'ai cherry-pické tous mes commits et les ai envoyés sur le staging (bon, j'ai pushé directement sur le gh-pages sans faire de PR #ThugLife).

http://staging.brownbaglunch.fr/baggers.html

[dadoonet]

Rhooo. C'est magnifique !

[dadoonet]

Comme on dit... Les cordonniers...

[dadoonet]

Petite remarque. On n'affichait pas l'email du speaker précédemment. Ca ne me gène pas plus que ça mais qu'en pensent les autres ?

[linsolas]

Je sais, certaines images ne passent pas (celles hébergées par LinkedIn par exemple). Il me semblait pourtant que la mienne était OK ! Je regarderais ce WEsinln...

[linsolas]

À voir. D'ailleurs je crois que l'envoi du mail foire (dans sa version actuelle). J'ai par exemple reçu SIX fois un mail d'invitation à un BBL cette semaine. Je suis le seul dans ce cas ?

[dadoonet]

Je suis le seul dans ce cas ?

Non. Ca arrive souvent. En fait, je pense que comme les gens n'ont pas spécialement de feedback (genre un email pour dire : nous avons bien transmis votre demande), ils réessayent.

C'est vrai que ça manque de feedback... :D

[linsolas]

Bah non, ca envoie le mail aussi à la personne qui invite (c'est pour ca qu'on demande l'email). Mais la, j'ai reçu les 6 mails en même temps, donc vrai souci du serveur...

[linsolas]

Pardon, je viens de vérifier, c'était 7 mails, pas 6 ;)

[linsolas]

@dadoonet Je viens de regarder sur la prod, le mail apparait bien dans la modale de contact (éditable aussi) :

[dadoonet]

My bad then. :(

[linsolas]

Et hop, petite mise à jour sur le staging, avec un logo animé dans le header, une image par défaut, et un loader (mais qui marche pas trop pour l'instant)...

[dadoonet]

Ca me semble vraiment génial @linsolas. Pour ceux qui voudraient tester: http://staging.brownbaglunch.fr/baggers.html

Pour moi, c'est beaucoup mieux qu'avant donc même si il y a des petits bugs ici ou là, je suis +1 pour merger ça.

[nrichand]

J'avais peu suivi (sorry 😓 ) mais un très grand bravo Romain c'est vraiment top. Je plussois l'avis de David, en l'état c'est beaucoup mieux que l'actuel donc je suis pour redescendre ça. Cette semaine j'ai poussé les BBL à mon équipe et je leur ai poussé l'url staging plutôt que la principale 😛

Pour le mailer je pense qu'il y a un bug en effet mais je ne sais pas ou (sur le web ou sur le mailer node...)

Encore un grand 👍 @linsolas

[linsolas]

Hello. Je remets d'équerre la PR (celle du staging à 2 - 3 commits d'avance) durant le WE, et go en prod. Vous me direz quels bugs vous avez trouvés (hormis celui du mail, qui ne semble pas être côté front ?) ?

[dadoonet]

Le seul bug que j'ai vu était "ta photo" et celle de quelques autres. Rien vu après tes derniers fix.

[linsolas]

Il y a un certain nombre de profils avec des photos foireuses, même avec le site actuel, en effet !

[fsarradin]

MA.GNI.FIQUE!!! ✨

Merci Romain :)

[linsolas]

Et hop, ça doit être synchro avec staging maintenant...

[nrichand]

Prêt à merger vers gh-pages ?

[linsolas]

moi je suis prêt

[dadoonet]

Aucun veto! Pousse le code! :)

[nrichand]

Goooooooooooo ! 👍

[linsolas]

Alors Gooooooooooo !

[linsolas]

C'est à moi de cliquer sur le clic ?

[nrichand]

Bah ouaip, on t'attend avec impatience, on en peu plus...

[linsolas]

ah ok. Je prépare ma souris pour le clic fatal

[linsolas]

ET PAF LE CHIEN

[nrichand]

🚀 🌕