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.

Sign up for GitHub

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

Jump to bottom

chore: Migration des pages flatpage vers wagtail #1496

Merged
merged 12 commits into from
Nov 20, 2024
Merged

chore: Migration des pages flatpage vers wagtail #1496

merged 12 commits into from
Nov 20, 2024

Conversation

chloend
Copy link
Contributor

@chloend chloend commented Nov 5, 2024

Quoi ?

Migration des pages FlatPage vers Wagtail.

Pourquoi ?

Pour regrouper toutes les pages dans le cms et éviter les confusions entre les modèles Page de FlatPage et Wagtail.

Comment ?

  • En supprimant le dossier 'lemarche/pages' qui comprenait les modèles Page et PageFragment.
  • En supprimant les pages inutiles dans 'lemarche/www/pages'.
  • En créant une commande de management pour populer le cms de pages statiques telles que 'cgu' ou 'mentions-legales'.
  • En créant les anciennes pages FlatPage dans wagtail, en production.

Autre (optionnel)

  • le fichier 'cms_content_pages.json' a été placé dans 'lemarche/fixtures' au lieu de 'lemarche/fixtures/django' parce qu'il sert à la commande create_content_pages et ne peut être chargé en tant que fixtures.

@chloend chloend changed the title Chloend/migration flatpage wagtail chore: Migration des pages flatpage vers wagtail Nov 5, 2024
@madjid-asa madjid-asa changed the base branch from master to chloend/sitemap November 7, 2024 16:23
@madjid-asa madjid-asa changed the base branch from chloend/sitemap to master November 7, 2024 16:25
madjid-asa
madjid-asa requested changes Nov 7, 2024
View reviewed changes
Copy link
Contributor

@madjid-asa madjid-asa left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Je pense que t'as du faire un oubli pour l'ajout du champ dans les pages :)
Mais sinon bien joué :D

lemarche/cms/models.py
@@ -214,7 +214,18 @@ class HomePage(Page):
use_json_field=True,
)

sub_header_custom_message = StreamField(
Copy link
Contributor

Choose a reason for hiding this comment

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

Pourquoi t'as ajouté ça ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

C'est pour remplacer l'utilisation du modèle PageFragment dans ArticleBase. Sorry j'ai oublié de préciser, j'avais oublié pourquoi j'avais ajouté ça en vrai héhé

Copy link
Contributor

Choose a reason for hiding this comment

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

Bien joué ;)

Copy link
Contributor

Choose a reason for hiding this comment

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

Le mettre juste en charfield sinon :)

lemarche/cms/migrations/0014_homepage_sub_header_custom_message.py Outdated
@@ -0,0 +1,25 @@
# Generated by Django 4.2.15 on 2024-10-23 10:31

from django.db import migrations
Copy link
Contributor

Choose a reason for hiding this comment

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

Je pense que t'as pas vraiment besoin de ça :)

SebastienReuiller
SebastienReuiller requested changes Nov 12, 2024
View reviewed changes
Copy link
Contributor

@SebastienReuiller SebastienReuiller left a comment

Choose a reason for hiding this comment

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

Merci @chloend d'avoir pris ce sujet.

Nous n'avons plus de page /faq, c'est voulu ?

Pour la suppression du module Pages, si on supprime d'un coup, nous allons avoir des tables résiduelles en base de données. Je te propose donc, de le "vider", notamment au niveau des modèles afin de générer les migrations de suppression. Nous pourrons le supprimer totalement dans une autre PR.

Aussi, je n'ai pas compris le sub_header_custom_message 🤔

madjid-asa
madjid-asa approved these changes Nov 13, 2024
View reviewed changes
Copy link
Contributor

@madjid-asa madjid-asa left a comment

Choose a reason for hiding this comment

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

LG 🗡️

lemarche/cms/models.py
@@ -214,7 +214,18 @@ class HomePage(Page):
use_json_field=True,
)

sub_header_custom_message = StreamField(
Copy link
Contributor

Choose a reason for hiding this comment

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

Bien joué ;)

madjid-asa
madjid-asa requested changes Nov 13, 2024
View reviewed changes
Copy link
Contributor

@madjid-asa madjid-asa left a comment

Choose a reason for hiding this comment

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

Nous n'avons plus de page /faq, c'est voulu ?

Elle est identique à la page Qui sommes nous ?

Pour la suppression du module Pages, si on supprime d'un coup, nous allons avoir des tables résiduelles en base de données. Je te propose donc, de le "vider", notamment au niveau des modèles afin de générer les migrations de suppression. Nous pourrons le supprimer totalement dans une autre PR.

Damn j'avais zappé toute les pages qu'il y avait dans ce dossier ^^
image

dont la méthode csrf qui est assez importante ^^ (malheureusement)

ce que tu peux supprimer c'est le fichier models de page, au niveau des vues c'est HomeView, PageView et c'est tout je crois, et au niveau des urls c'est toutes celles qui sont basées sur HomeView (path("ancien_accueil") et PageView, et toute les statics (de la ligne 36 à 65) :visage_légèrement_souriant:

Aussi, je n'ai pas compris le sub_header_custom_message 🤔
C'est pour les messages de promotion on les utilisaient à un moment notamment pour les annonces de webinaires

@chloend chloend added the recette-jetable Déclenche une recette jetable sur CC label Nov 19, 2024
@chloend chloend force-pushed the chloend/migration-flatpage-wagtail branch from 2e2a3ff to a8255de Compare November 19, 2024 18:42
@SebastienReuiller SebastienReuiller added recette-jetable Déclenche une recette jetable sur CC and removed recette-jetable Déclenche une recette jetable sur CC labels Nov 20, 2024
@github-actions GitHub Actions
Copy link
Contributor

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@chloend chloend added recette-jetable Déclenche une recette jetable sur CC and removed recette-jetable Déclenche une recette jetable sur CC labels Nov 20, 2024
@github-actions GitHub Actions
Copy link
Contributor

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@chloend chloend merged commit ac01f78 into master Nov 20, 2024
10 of 11 checks passed
@chloend chloend deleted the chloend/migration-flatpage-wagtail branch November 20, 2024 17:20
chloend added a commit that referenced this pull request Nov 25, 2024
@chloend
chore: Migration des pages flatpage vers wagtail (#1496)
8c1ce41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SebastienReuiller SebastienReuiller SebastienReuiller requested changes

@madjid-asa madjid-asa madjid-asa requested changes

Assignees

@chloend chloend

Labels
recette-jetable Déclenche une recette jetable sur CC
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@chloend @madjid-asa @SebastienReuiller