-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat(forum): attr accept définit sur forms.ImageField #734
Conversation
…s d'image courants
lacommunaute/forum/forms.py
Outdated
image = forms.ImageField( | ||
required=False, | ||
label="Banniere de couverture", | ||
widget=forms.FileInput(attrs={"accept": settings.SUPPORTED_IMAGE_FILE_TYPES.keys()}), | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Attention qu'on accepte toujours les autres fichiers acceptés par ImageField
(TIFF, WEBP), je n'ai pas ajouté une contrainte sur la base de données. Si on décide que c'est une contrainte forte, on pourrait en ajouter
Je n'ai pas enquêter s'il y a des images obscures sur prod qui on aurait besoin de transformer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎯 excellent !
🤖 I have created a release *beep* *boop* --- ## [2.12.0](v2.11.0...v2.12.0) (2024-08-07) ### Features * **documentation:** rendre la certification optionnelle ([#741](#741)) ([6141112](6141112)) * **footer:** mise à jour des liens sociaux ([#730](#730)) ([f19602e](f19602e)) * **forum:** attr accept définit sur forms.ImageField ([#734](#734)) ([0e8b8c9](0e8b8c9)) * **forum:** fin de l'AB test sur la notation des forums ([#708](#708)) ([6705ead](6705ead)) * **forum:** utiliser l'image du forum dans la vignette opengraph ([#725](#725)) ([cc137c1](cc137c1)) * mise à jour entête, pied de page et menu ([#739](#739)) ([4493267](4493267)) * **notification:** passage des notifications en lues ([#712](#712)) ([a1a6b73](a1a6b73)) * **seo:** mise à jour de l'image OpenGraph ([#720](#720)) ([f37521c](f37521c)) ### Bug Fixes * **forum:** balise `article` et dimension des images ([#737](#737)) ([b1ed91f](b1ed91f)) * mise à jour de la public key sentry pour l'envoi des logs ([#732](#732)) ([616c0aa](616c0aa)) * **upvotes:** largeur du bouton s'abonner dynamique ([#743](#743)) ([32cb339](32cb339)) * **upvote:** supprimer un parametrage ambigue ([#742](#742)) ([168df13](168df13)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Quelque chose que je viens d'implementer sur les-emplois, j'ai vu l'occasion de repliquer le travail sur la communauté :)
Description
🎸 Ajouté une définition des formats d'image acceptés (encouragés) sur les thématiques
🎸 Ajouté un champ dans le settings du projet pour clarifier les formats d'image soutenu. J'ai choisis les formats d'image qui sont soutenu par le plupart des navigateurs mondiales
Type de changement
🎨 changement d'UI
Points d'attention
🦺 J'ai inclus GIF 😁