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

Indiquer la taille des fichiers téléchargeables #872

Closed
Eskimon opened this issue Jun 12, 2014 · 7 comments

Comments

@Eskimon
Copy link
Member

commented Jun 12, 2014

Via Jeekwiz

C'est qu'un détail mais ce serait bien d'afficher la taille des fichiers téléchargeables à coté du lien (PDF, HTML, EPUB, Archive).

@Eskimon Eskimon added Front labels Jun 12, 2014

@SpaceFox

This comment has been minimized.

Copy link
Member

commented Jun 12, 2014

Si tant est qu'on dispose de cette taille à priori, ce dont je ne suis pas
certain.

Le 12 juin 2014 15:11, Eskimon notifications@github.com a écrit :

Via Jeekwiz
http://zestedesavoir.com/forums/sujet/268/indiquer-la-taille-des-fichiers-telechargeables/

C'est qu'un détail mais ce serait bien d'afficher la taille des fichiers
téléchargeables à coté du lien (PDF, HTML, EPUB, Archive).


Reply to this email directly or view it on GitHub
#872.

@Eskimon

This comment has been minimized.

Copy link
Member Author

commented Jun 12, 2014

Google me dit : os.path.getsize(path)

Apres je sais pas si les liens sont en BDD ou pas etc... car ca pourrait etre judicieux d'ajouter un champ pour la taille du fichier pour eviter de la recalculer a chaque affichage...)

@Alex-D Alex-D added the Back label Jun 12, 2014

@Alex-D

This comment has been minimized.

Copy link
Contributor

commented Jun 12, 2014

Les fichiers sont générés à la validation, donc on peut mesurer leur taille :)

@ArnaudCalmettes ArnaudCalmettes added this to the Version 2.0+ milestone Jun 13, 2014

@SpaceFox SpaceFox removed the Back label Jul 9, 2014

@SpaceFox SpaceFox modified the milestones: Version v1.x, Version 2.0+ Jul 9, 2014

@gustavi gustavi added the C-Back label Nov 30, 2014

@Eskimon Eskimon removed the C-Front label Apr 21, 2015

@Eskimon Eskimon removed this from the "Futur proche" (v1.x) milestone Apr 21, 2015

@SpaceFox

This comment has been minimized.

Copy link
Member

commented Oct 1, 2015

Cette idée n'a pas l'air de motiver quiconque depuis plus d'un an. Je la considère donc comme abandonnée, si ça intéresse quelqu'un, il suffit de ré-ouvrir.

@SpaceFox SpaceFox closed this Oct 1, 2015

@pierre-24

This comment has been minimized.

Copy link
Member

commented Oct 2, 2015

Raaah, c'est con, ça aurait pris 30 secondes à faire si je l'avais su ^^

@pierre-24 pierre-24 reopened this Oct 2, 2015

@pierre-24 pierre-24 self-assigned this Oct 2, 2015

@gustavi

This comment has been minimized.

Copy link
Member

commented Nov 21, 2015

@pierre-24 je peux m'occuper de ça si tu n'as pas commencé ?

@pierre-24

This comment has been minimized.

Copy link
Member

commented Nov 21, 2015

Je t'en prie ;)

@gustavi gustavi closed this in 891394e Dec 18, 2015

pierre-24 added a commit that referenced this issue Dec 18, 2015
Merge pull request #3224 from gustavi/fix-872
Fix #872 : ajout de la taille des fichiers telecharges

@pierre-24 pierre-24 added this to the Version de développement milestone Dec 18, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.