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

Permettre aux admins d'héberger des fichiers #447

Merged
merged 3 commits into from
Oct 8, 2024
Merged

Conversation

thibault
Copy link
Collaborator

@thibault thibault commented Oct 7, 2024

https://trello.com/c/mrVJ66YO/1164-permettre-aux-admin-dh%C3%A9berger-des-pdf-en-autonomie

Je n'ai pas limité l'upload aux fichiers pdfs, au cas ou…

@thibault thibault requested a review from pyDez October 7, 2024 13:19
Copy link
Collaborator

@pyDez pyDez left a comment

Choose a reason for hiding this comment

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

On ne fait aucune validation sur le fichier, s'il nous envoie un fichier de 1To, ou un executable, on l'accepte. Ce n'est pas forcement bloquant, mais a garder en tête.

Question annexe, tu as une methode pour générer les traductions des champs des models ?

@thibault
Copy link
Collaborator Author

thibault commented Oct 8, 2024

@pyDez

On ne fait aucune validation sur le fichier, s'il nous envoie un fichier de 1To, ou un executable, on l'accepte. Ce n'est pas forcement bloquant, mais a garder en tête.

Tu as raison, c'est peut-être un peu cavalier. J'ai ajouté une validation sur la taille max de fichier, et j'ai sélectionné 10mo pour le moment, ce qui devrait être suffisant.

En revanche, dans la mesure ou c'est réservé aux admins, je me suis dit qu'on n'aurait pas de problème même en ne sécurisant pas le type de fichier. Ça te va ?

Question annexe, tu as une methode pour générer les traductions des champs des models ?

Désolé je n'ai pas compris ta question. Tu peux reformuler ?

@pyDez
Copy link
Collaborator

pyDez commented Oct 8, 2024

@thibault

En revanche, dans la mesure ou c'est réservé aux admins, je me suis dit qu'on n'aurait pas de problème même en ne sécurisant pas le type de fichier. Ça te va ?

Ca me va

Question annexe, tu as une methode pour générer les traductions des champs des models ?

Désolé je n'ai pas compris ta question. Tu peux reformuler ?

Ca n'est pas lié à la validation de la PR. J'ai juste remarqué que tu avais généré des traductions pour les noms des champs dans le fichier django.po. Je voulais savoir comment tu faisais cela ? Ton ide ? un outils tiers ? A la main ?

@thibault
Copy link
Collaborator Author

thibault commented Oct 8, 2024

@pyDez

Ca n'est pas lié à la validation de la PR. J'ai juste remarqué que tu avais généré des traductions pour les noms des champs dans le fichier django.po. Je voulais savoir comment tu faisais cela ? Ton ide ? un outils tiers ? A la main ?

J'ai juste lancé la commande makemessages. Tu vois des choses dans le fichier qui ne devraient pas y être ?

@thibault thibault merged commit 4e5acdf into main Oct 8, 2024
4 checks passed
@thibault thibault deleted the admin_uploads branch October 8, 2024 13:30
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