search

hostolab / covidliste

Speed-up Covid-19 vaccination campaigns 💉
https://www.covidliste.com
MIT License
255 stars 55 forks source link

Convertir les vues HAML en ERB #167

Closed guillaumewrobel closed 3 years ago

guillaumewrobel commented 3 years ago
  1. [x] Avant : merger les PR qui modifient beaucoup de fichiers et qui souffriraient d'un rebase
  2. [x] Faire une PR de conversion, s'y mettre à plusieurs pour traiter toutes les vues d'un coup
  3. [x] Merger sur master
mathieuripert commented 3 years ago

J'ai pas bien suivi mais j'ai l'impression qu'on reste en HAML finalement, non ? https://covidliste.slack.com/archives/C01T48BHRCL/p1617731640223400

guillaumewrobel commented 3 years ago

J'ai pas bien suivi mais j'ai l'impression qu'on reste en HAML finalement, non ? https://covidliste.slack.com/archives/C01T48BHRCL/p1617731640223400

Justement, je suis seul contre tous :smile: mais c'est plus grand-public de rester sur ERB.

mathieuripert commented 3 years ago

J'ai pas bien suivi mais j'ai l'impression qu'on reste en HAML finalement, non ? https://covidliste.slack.com/archives/C01T48BHRCL/p1617731640223400

au temps pour moi, j'ai mal lu :)

mathieuripert commented 3 years ago

J'ai pas bien suivi mais j'ai l'impression qu'on reste en HAML finalement, non ? https://covidliste.slack.com/archives/C01T48BHRCL/p1617731640223400

Justement, je suis seul contre tous 😄

j'ai voté pour le mauvais :)

ssaunier commented 3 years ago

Il en reste encore un peu 👇 

$ find . -type f -name "*.haml"

./app/views/admin/search/search.html.haml
./app/views/admin/vaccination_centers/show.html.haml
./app/views/admin/vaccination_centers/index.html.haml
./app/views/admin/vaccination_centers/edit.html.haml
./app/views/admin/vaccination_centers/new.html.haml
./app/views/mailer/_styles.html.haml
./app/views/layouts/_navbar.html.haml
./app/views/layouts/_admin_navbar.html.haml
./app/views/partners/sessions/new.html.haml
./app/views/partners/vaccination_centers/show.html.haml
./app/views/partners/vaccination_centers/index.html.haml
./app/views/partners/vaccination_centers/new.html.haml
./app/views/partners/campaigns/show.html.haml
./app/views/partners/campaigns/new.html.haml
./app/views/partners/new.html.haml
./lib/templates/haml/scaffold/_form.html.haml
colinemarie commented 3 years ago

Je veux bien en convertir. @mininao tu n'es pas sur celles de partner pour ton design ?

guillaumewrobel commented 3 years ago

Finito https://github.com/hostolab/covidliste/pull/398 :muscle: