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

[Ne pas merger] Début de mise à jour du template PaperMod #11

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

thbar
Copy link
Contributor

@thbar thbar commented May 10, 2023

Voir #10.

J'ai l'impression en comparant sur des fichiers que ça casse pas mal de choses. Il faudra passer en revue, mais ça fait une première base de travail.

EDIT : en regardant la preview ça semble bon par contre, en comparant avec la production, à première vue aussi.

Différences

Je ne suis pas sûr de mon calcul, il peut y avoir des restes qui traînaient (j'ai pu oublier de purger un élément ou un autre, ou mal mettre à jour un submodule), donc à revisiter.

CleanShot 2023-05-10 at 18 09 32@2x

@netlify
Copy link

netlify bot commented May 10, 2023

Deploy Preview for transport-normes ready!

Name Link
🔨 Latest commit ba1ec05
🔍 Latest deploy log https://app.netlify.com/sites/transport-normes/deploys/645bc19b0cf82400087b8b0f
😎 Deploy Preview https://deploy-preview-11--transport-normes.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.

@fchabouis
Copy link
Contributor

Merci @thbar !
En me promenant sur le site de preview, je n'ai pas vu de problème.
Il faut savoir que j'ai copié certains fichiers du theme papermode dans notre dossier layouts pour les customizer. Ces layouts ont la priorité sur les layouts du thème. Mais ils ne sont pas mis à jour par cette PR. Je ne suis pas sûr que ce soit gênant, mais ça veut dire que la mise à jour du thème est partielle du coup. J'avoue que je ne sais pas quelle est la meilleure manière de faire ça. Est-ce qu'on devrait recopier les nouveaux templates dans le dossier template et re-merger nos customizations dedans ?

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