New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Day 7 #24

Merged
merged 5 commits into from Dec 7, 2017

Conversation

Projects
None yet
4 participants
@oncletom
Member

oncletom commented Dec 4, 2017

(in progress)

@davidbgk

This comment has been minimized.

Show comment
Hide comment
@davidbgk

davidbgk Dec 5, 2017

Member

Peut-être que l'on pourrait faire quelque chose de spécial le 7 vu que ça fêtera les 10 ans de l'opendata https://opengovdata.org/ ?

Member

davidbgk commented Dec 5, 2017

Peut-être que l'on pourrait faire quelque chose de spécial le 7 vu que ça fêtera les 10 ans de l'opendata https://opengovdata.org/ ?

@oncletom

This comment has been minimized.

Show comment
Hide comment
@oncletom

oncletom Dec 5, 2017

Member

Yes pas de prob — y'a pas d'impératif à ce que ce contenu soit publié ce jour là :-)

Member

oncletom commented Dec 5, 2017

Yes pas de prob — y'a pas d'impératif à ce que ce contenu soit publié ce jour là :-)

@oncletom oncletom changed the title from Day 7 to Day 8 Dec 6, 2017

oncletom and others added some commits Dec 4, 2017

@oncletom oncletom changed the title from Day 8 to Day 7 Dec 7, 2017

@oncletom

This comment has been minimized.

Show comment
Hide comment
@oncletom

oncletom Dec 7, 2017

Member

Il manque 2-3 trucs pour merger :

  • le code source de la dataviz Loire-Atlantique
  • le code source/lien de de la dataviz Côtes d’Armor
  • le code source/lien de la dataviz Haute-Garonne

Dans le pire des cas, on les mentionne sans les liens et on les rajoute quand on les a.

Member

oncletom commented Dec 7, 2017

Il manque 2-3 trucs pour merger :

  • le code source de la dataviz Loire-Atlantique
  • le code source/lien de de la dataviz Côtes d’Armor
  • le code source/lien de la dataviz Haute-Garonne

Dans le pire des cas, on les mentionne sans les liens et on les rajoute quand on les a.

@pachevalier

This comment has been minimized.

Show comment
Hide comment
@pachevalier

pachevalier Dec 7, 2017

Contributor

Sur côte d'Armor, j'ai trouvé que ça et c'est pas très détaillé. Je propose de suppprimer.

Pour Loire Atlantique, j'ai trouvé une dataviz agrégée mais pas des données détaillées.

Contributor

pachevalier commented Dec 7, 2017

Sur côte d'Armor, j'ai trouvé que ça et c'est pas très détaillé. Je propose de suppprimer.

Pour Loire Atlantique, j'ai trouvé une dataviz agrégée mais pas des données détaillées.

pachevalier added some commits Dec 7, 2017

@oncletom

This comment has been minimized.

Show comment
Hide comment
@oncletom

oncletom Dec 7, 2017

Member

Aucune n'a les données détaillées, ni un lien vers les jeux de données :-(

Member

oncletom commented Dec 7, 2017

Aucune n'a les données détaillées, ni un lien vers les jeux de données :-(

@@ -0,0 +1,46 @@
---
title: Dataviz Finances Gironde
advent_day: 2017-12-07

This comment has been minimized.

@oncletom

oncletom Dec 7, 2017

Member

@pachevalier c'est plus nécessaire :-) c'était un contournement utilisé au début mais que j'ai réglé avec le support de Jekyll (cf. jekyll/jekyll#6581)

@oncletom

oncletom Dec 7, 2017

Member

@pachevalier c'est plus nécessaire :-) c'était un contournement utilisé au début mais que j'ai réglé avec le support de Jekyll (cf. jekyll/jekyll#6581)

This comment has been minimized.

@pachevalier

pachevalier Dec 7, 2017

Contributor

ok

@pachevalier

pachevalier Dec 7, 2017

Contributor

ok

This comment has been minimized.

@pachevalier

pachevalier Dec 7, 2017

Contributor

Mais pourquoi ça n'apparaît pas sur la case 7 https://deploy-preview-24--eloquent-mirzakhani-50c5e9.netlify.com/ ?

@pachevalier

pachevalier Dec 7, 2017

Contributor

Mais pourquoi ça n'apparaît pas sur la case 7 https://deploy-preview-24--eloquent-mirzakhani-50c5e9.netlify.com/ ?

@pachevalier pachevalier merged commit 5bccf20 into master Dec 7, 2017

4 checks passed

ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
deploy/netlify Deploy preview ready!
Details

@oncletom oncletom deleted the day-07 branch Dec 7, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment