Skip to content
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

Article sur les process data OSM chez Geovelo #472

Merged
merged 35 commits into from
Nov 26, 2021
Merged

Conversation

samDB-gv
Copy link
Contributor

Bonjour,

Une première version d'un article sur la gestion des données OSM chez Geovelo (il manque quelques images, je n'ai pas les droits pour les stocker sur le CDN de Geotribu :p )

N'hésitez pas s'il y a des modifications/eclaircissements/suppressions à faire !

Merci et bonne journée

@github-actions github-actions bot added articles Tutoriels, actualité, billet... contributeurices Contenus liés à la contribution et contributeur/ices : biographies, à propos, etc. labels Nov 15, 2021
@Guts Guts self-assigned this Nov 15, 2021
@Guts Guts added the contribution externe Contenus proposés par des personnes n'ayant pas de droits en écriture sur le dépôt GitHub label Nov 15, 2021
@Guts Guts marked this pull request as draft November 16, 2021 10:43
Copy link
Member

@Guts Guts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci de ne pas modifier autre chose que les markdown liés à l'article :)

@Guts
Copy link
Member

Guts commented Nov 17, 2021

Premier jet.

Merci de ne pas modifier autre chose que les markdown liés à l'article :)

J'ai donc annulé la modif du package.json.
J'ai ajouté ton mail au mailmap pour que ce soit plus joli.

Attention, le fichier de ta bio a 5 lettres au lieu des 4.

Sinon, j'ai juste appliqué les corrections automatiques du linter : yarn markdownlint -f content/articles/2021/2021-11-20_traitements_data_osm_geovelo.md

@samueldb
Copy link

Merci pour les corrections et désolé pour le package.json qui m'a échappé :$

@igeofr
Copy link
Contributor

igeofr commented Nov 22, 2021

Bonjour @samueldb,
J'ai fait une relecture de l'article et j'en ai profité pour faire quelques corrections sur le rendu.

  • Pour l’image de l’en-tête notamment utilisée pour la page d’accueil, est-ce que c'est possible de pousser sur le cdn une image horizontale aux dimensions approximative de 1000x600px?
  • Une remarque à discuter, avec @Guts et @Jersig, je trouve que la citation markdown est trop utilisée

Copy link
Member

@Guts Guts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Première passe sur la syntaxe globale et les coquilles typos.

@Jersig
Copy link
Contributor

Jersig commented Nov 24, 2021

Bonjour @samueldb,
Merci pour cet article, j'ai jeté un œil, j'effectue une relecture plus complète vendredi matin pour compléter celle d'@igeofr

@Guts Guts marked this pull request as ready for review November 26, 2021 09:29
@github-actions github-actions bot added the thème Personnalisation du thème du site (CSS, JS, partials...) label Nov 26, 2021
@github-actions
Copy link
Contributor

🎉 Le site de prévisualisation pour cette PR a été généré et est disponible sous ce lien.
Commit de référence : 311d01e.

@Guts Guts merged commit 8903c0e into geotribu:master Nov 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
articles Tutoriels, actualité, billet... contributeurices Contenus liés à la contribution et contributeur/ices : biographies, à propos, etc. contribution externe Contenus proposés par des personnes n'ayant pas de droits en écriture sur le dépôt GitHub thème Personnalisation du thème du site (CSS, JS, partials...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants