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

[CMS- Homepage] Ajouter la possibilité aux admins de modifier les images #906

Merged
merged 1 commit into from
Sep 20, 2023

Conversation

madjid-asa
Copy link
Contributor

Quoi ?

Ajouter la possibilité aux admins de modifier les images.

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.

Juste une petite remarque sur l'attribut alt.
Je t'aurais bien suggéré de supprimer les images de static, mais elles sont encore utiliser sur l'ancienne home. D'ailleurs, pourquoi on la conserve ?
Autre truc, est-ce que tu sais comment on peut charger des champs images dans les fixtures ? Là par exemple, comment on pourrait ajouter des partenaires à la fixture "home page"?

lemarche/cms/blocks.py Show resolved Hide resolved
@madjid-asa madjid-asa force-pushed the madjid-asa/cmsify-section-our-partners branch from 7aadf40 to e8195b5 Compare September 18, 2023 14:52
@madjid-asa
Copy link
Contributor Author

Juste une petite remarque sur l'attribut alt. Je t'aurais bien suggéré de supprimer les images de static, mais elles sont encore utiliser sur l'ancienne home. D'ailleurs, pourquoi on la conserve ? Autre truc, est-ce que tu sais comment on peut charger des champs images dans les fixtures ? Là par exemple, comment on pourrait ajouter des partenaires à la fixture "home page"?

Non pour les images dans fixtures je sais pas trop comment faire, je pense qu'il faudrait le scripter, mais comme en prod, recette jetable et local c'est pas la même gestion, on l'a jamais priorisé.

@madjid-asa madjid-asa merged commit 3839a37 into master Sep 20, 2023
5 checks passed
@madjid-asa madjid-asa deleted the madjid-asa/cmsify-section-our-partners branch September 20, 2023 09:53
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