Vuejs

[linsolas]

No description provided.

[linsolas]

PR faisant suite à l'issue #341
Quelques points à voir / corriger :

• éviter d'avoir un affichage avec des placeholders en place tant que les données ne sont pas récupérées.
• 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]

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]

🚀 🌕