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

Add blog #1091

Merged
merged 10 commits into from
Jun 4, 2023
Merged

Add blog #1091

merged 10 commits into from
Jun 4, 2023

Conversation

dxb
Copy link
Collaborator

@dxb dxb commented Jun 1, 2023

  • ajouter une page qui liste les billets
  • ajouter une page qui affiche un billet
  • ajouter des billets en markdown
  • ajouter un lien dans le footer
  • ajouter des tests
  • améliorer l'affichage
  • améliorer la structure

@netlify
Copy link

netlify bot commented Jun 1, 2023

Deploy Preview for nosgestesclimat ready!

Name Link
🔨 Latest commit a10a53a
🔍 Latest deploy log https://app.netlify.com/sites/nosgestesclimat/deploys/647a414523b1650008a99a68
😎 Deploy Preview https://deploy-preview-1091--nosgestesclimat.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions
Copy link

github-actions bot commented Jun 1, 2023

Report for the pull request #1091


🌐 Translation status

UI's texts

Language Nb. missing translations Status
en-us
Missing 1 rules ⬇️
    Blog,Blog

FAQ's questions

Language Nb. missing translations Status
en-us Ø ✔️

You will find more information about the translation in the dedicated file.

@dxb dxb marked this pull request as ready for review June 1, 2023 14:18
@cypress
Copy link

cypress bot commented Jun 1, 2023

Passing run #277 ↗︎

0 24 0 0 Flakiness 0

Details:

Merge a10a53a into 4384c4b...
Project: Nos Gestes Climat Commit: e838fa5c0f ℹ️
Status: Passed Duration: 02:27 💡
Started: Jun 2, 2023 7:22 PM Ended: Jun 2, 2023 7:25 PM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@laem
Copy link
Contributor

laem commented Jun 1, 2023

@dxb je ne sais pas à quel poitn c'est nécessaire pour la v1 (à mon avis non), mais il faudra introduire des attributs dans les .md.

Ça se fait en général avec front-matter, et les parsers markdown gère ça avec un plugin. Pour la date d'écriture, l'image de couverture, le sous-titre, les auteurs, etc.

CF cet article https://github.com/laem/blog/blob/master/articles/versez-votre-acompte-%C3%A9cologique.md
Et ce composant Article sur mon blog perso https://github.com/laem/blog/blob/master/Article.js#L55

J'ai ajouté une feature cool aussi : afficher la date de dernière modification de l'article via l'API github :)

https://github.com/laem/blog/blob/master/Article.js#L13

En tout cas, trop cool d'avoir déjà quelque chose :)

@laem laem mentioned this pull request Jun 1, 2023
[Warning] Certains liens ne fonctionnent pas : il faut les nouvelles URL internes.
@JuliePouliquen
Copy link
Contributor

Génial @laem je demandais justement à @dxb comment ajouter les métadonnées. Sur notre site perso en construction, on a aussi frontmatter, donc je vois bien l'idée et c'est top !

@dxb
Copy link
Collaborator Author

dxb commented Jun 2, 2023

@dxb je ne sais pas à quel poitn c'est nécessaire pour la v1 (à mon avis non), mais il faudra introduire des attributs dans les .md.

Ça se fait en général avec front-matter, et les parsers markdown gère ça avec un plugin. Pour la date d'écriture, l'image de couverture, le sous-titre, les auteurs, etc.

CF cet article https://github.com/laem/blog/blob/master/articles/versez-votre-acompte-%C3%A9cologique.md Et ce composant Article sur mon blog perso https://github.com/laem/blog/blob/master/Article.js#L55

J'ai ajouté une feature cool aussi : afficher la date de dernière modification de l'article via l'API github :)

https://github.com/laem/blog/blob/master/Article.js#L13

En tout cas, trop cool d'avoir déjà quelque chose :)

Je suis ok avec ça

@dxb dxb closed this Jun 2, 2023
@dxb dxb reopened this Jun 2, 2023
@dxb dxb merged commit c5b490d into master Jun 4, 2023
@dxb dxb deleted the add-blog branch June 4, 2023 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants