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

Make blog static #487

Merged
merged 10 commits into from
Jan 15, 2025
Merged

Make blog static #487

merged 10 commits into from
Jan 15, 2025

Conversation

flacombe
Copy link
Contributor

@flacombe flacombe commented Jan 8, 2025

A la brutale, on enregistre les deux json issus de ghost et on abstrait l'API dans blog-static.

J'espère que ca marche, je n'ai pas encore testé.

Aurais-tu un avis @GllmR ?

@flacombe flacombe requested a review from GllmR January 8, 2025 16:11
@GllmR
Copy link
Contributor

GllmR commented Jan 8, 2025

Salut !

Je ne suis pas sûr de comprendre ce que tu veux faire.

Tu voudrais virer le Ghost et faire les articles "à la main" ?

Je pense que ça devrait fonctionner comme tu as fait, je testerai en local dès que j'ai un peu de temps.

@flacombe
Copy link
Contributor Author

flacombe commented Jan 8, 2025

Oui, on supprime le ghost et on conserve le contenu en ligne. On ne fera pas plus d'articles malheureusement :(

@GllmR
Copy link
Contributor

GllmR commented Jan 8, 2025

Okay, je jette un œil ce soir ;)

@flacombe flacombe force-pushed the feature/staticblog branch from 61f0bff to f828767 Compare January 8, 2025 17:33
@flacombe
Copy link
Contributor Author

flacombe commented Jan 8, 2025

Merci beaucoup Guillaume !

J'ai finalisé le js et ajouté les images. Il n'y a normalement plus une référence au ghost.
On garde biensur l'intégration qui avait été faite de sorte à pouvoir restaurer un ghost, un jour peut-être.

@GllmR
Copy link
Contributor

GllmR commented Jan 8, 2025

Voilà @flacombe , ça devrait fonctionner. J'ai mis des trucs en commentaires, mais on pourrait les virer.

Il y a une chose qui reste à faire : Les images ne s'affichent pas dans les articles, parce qu'il faut virer l'adresse du site, comme par exemple ici (J'avoue j'ai pas le courage de fouiller pour retrouver les lien des images haha)

pages/index.js Outdated Show resolved Hide resolved
components/post.js Outdated Show resolved Hide resolved
@flacombe
Copy link
Contributor Author

flacombe commented Jan 8, 2025

Merci d'avoir regardé @GllmR
Je pense qu'on peut supprimer moins de trucs que ça, parce que je peux rajouter les auteurs dans le json (et modifier l'URL des images, bien sûr, ca je sais faire facilement)

@flacombe
Copy link
Contributor Author

flacombe commented Jan 8, 2025

Il y a un détail qui m'échappe sur la pagination :

  • Actuellement nous affichons 9 posts par page (cohérent avec lib/blog.js et LIMIT = 9)
  • La pagination que renvoi ghost dans le json est différente :
"pagination": {
    "page": 1,
    "limit": 15,
    "pages": 1,
    "total": 10,
    "next": null,
    "prev": null
}

Ainsi l'absence du bloc meta dans le json ne doit pas avoir beaucoup d'impact (sinon je peux l'ajouter, ce n'est pas grave)

@GllmR
Copy link
Contributor

GllmR commented Jan 10, 2025

Il y a un détail qui m'échappe sur la pagination :

* Actuellement nous affichons 9 posts par page (cohérent avec lib/blog.js et LIMIT = 9)

* La pagination que renvoi ghost dans le json est différente :
"pagination": {
    "page": 1,
    "limit": 15,
    "pages": 1,
    "total": 10,
    "next": null,
    "prev": null
}

Ainsi l'absence du bloc meta dans le json ne doit pas avoir beaucoup d'impact (sinon je peux l'ajouter, ce n'est pas grave)

Effectivement, je ne sais pas pourquoi, mais la pagination ne fonctionne pas.
Ceci dit, il n'y a que 10 articles à afficher.

flacombe and others added 8 commits January 14, 2025 23:37
Commentaires sur les parties qui utilisaient `getStaticProps`
Modifications pour l'affichages des images
Correction du truc pour les tags
Commentaire sur les parties qui affichent les auteur(e)s (pas dispo dans le json)
components/post.js Outdated Show resolved Hide resolved
@flacombe flacombe merged commit 757db45 into main Jan 15, 2025
1 check passed
@flacombe flacombe deleted the feature/staticblog branch January 15, 2025 09:31
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.

2 participants