Qwann/k fet/stats #443
No reviewers
Labels
No labels
devtype -- backend
devtype -- docs
devtype -- frontend
devtype -- user interface
difficulty -- easy
difficulty -- hard
difficulty -- normal
Doing
domain -- bda
domain -- bds
domain -- cof
domain -- core
domain -- kfet
Good first issue
priority -- high
priority -- low
priority -- medium
priority -- staff-wanted
status -- development
status -- discussion
status -- need review
status -- production
status -- ready to merge
status -- todo
To Do
type -- bug
type -- hygiene
type -- improvement
type -- new feature
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: DGNum/gestioCOF#443
Loading…
Reference in a new issue
No description provided.
Delete branch "qwann/k-fet/stats"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
New Features
Rajoute quelques graphiques :
Il y a déjà un outil pour faire ça (doc ici).
Ça donnerait
C'est parfaitement intégré à Django et ça gère les locales donc on n'a pas que le français.
Par ailleurs, ceci étant très court, je ne sais pas s'il est utile de faire une fonction auxiliaire.
Idem :
Idem
Seriously…
Ce filtre existe déjà dans
gestioncof/templatetags/utils
Je pense que certains de tes commentaires au dessus des classes ont leur places comme docstrings.
S'ils aident à utiliser correctement tes mixins et vues génériques, ce serait bien de les avoir quand on appelle
help
.""
est une bonne valeur par défaut aussi ;)Je pense que tes vues génériques et mixins devraient aller dans le fichier
kfet/statistics.py
. Ce sont des outils pour crafter des vues plutôt que des vues, ce serait (à mon sens) plus facile à trouver et ça allégerait le fichierviews.py
déjà bien lourd.Added 1 commit:
74f4d94f
- first fixesSauf erreur, cette fonction n'est jamais importée ; en tout cas je me prends un 500 à cause de ça.
Il me semble que tu as mis cette fonction dans un fichier js à part ; du coup il faudrait le sourcer plutot que de la réécrire.
Idem ici.
Ici aussi.
typo
Marked this merge request as a Work In Progress
Je chipote mais tu peux faire
names[i] = dates[i].strftime("Semaine %W")
pour éviter la concaténation.Typos :
Idem
typo intto
I think that "specified" is better than "precised"
Tu ne penses pas que tu devrais renvoyer un dictionnaire vide ou raise une erreur plutôt que de ne rien faire ?
J'ai bien compris que cette fonction était supposée être surchargée mais actuellement, si quelqu'un utilise mal cette classe et ne surcharge pas cette fonction il aura des erreurs étranges parce que le résultat de
get_dates
seraNone
. Au moins une erreur explicite serait plus claire.Réponse de @narmanli : de cette façon le code de l'appli
kfet
reste relativement indépendant du reste de GestioCOF.Ça me va tout à fait
Pas plus de remarques a priori. Maintenant je dois tester.
Trop de duplication dans
kfet/views.py
etkfet/urls.py
.Une url par type de stats devrait suffire puis n'utiliser que 1, 2 ou 3 classes de vues (selon le type de stats). L'intervalle de temps devient alors un paramètre de la classe et on a qqch (fonction ? dictionnaire ?) pour faire la conversion texte -> duration (ou date).
Accessoirement cette fonction a l'air assez générale, et son utilisation pourrait ne pas uniquement être limitée aux statistiques ; on pourrait directement la mettre dans
kfet.js
.Added 79 commits:
k-fet
5877218a
- first fixes76db0c86
- Add fixtures.a14f1db9
- doc added for kfet new fixtures1ddd34d5
- Merge branch 'qwann/k-fet/stats' of git.eleves.ens.fr:cof-geek/gestioCOF into qwann/k-fet/statsbdbb252a
- Revert "first fixes"7e5910e0
- balance templatetag fixed8b4f7ad6
- decimal cast for operation.amountfff7a202
- conflict solvedde3db140
- Merge branch 'qwann/decimal_fixes' into qwann/k-fet/statse6854d79
- using strftime instead of a personnal functionf8c49ae9
- docstring instead of comments232b2930
- typos fixed61a7f2df
- sourcing get_thing in a js fileAdded 1 commit:
947de3b6
- wrong months in 'last months' statsAdded 1 commit:
df2a9bb1
- factorising BalanceStat viewsUnmarked this merge request as a Work In Progress
On ne met plus ces choses là, cf #103.
Il faut donc enlever les
from __future__ import ...
et lesfrom builtins import *
Added 3 commits:
94f8705e
- 1 commit from branchk-fet
f48f6c57
- Merge branch 'qwann/decimal_fixes' into qwann/k-fet/stats9607fab0
- future removedÇa peut se faire avec l'ORM :
queryset.aggregate(Sum('article_nb'))['article_nb__sum']
c'est plus rapide et plus court.Added 1 commit:
bd7f7679
- CleanupJe maintiens que ce serait bien de déplacer la plomberie dans
statistics.py
pour allégerviews.py
.Added 10 commits:
k-fet
a13c4073
- Merge branch 'k-fet' of git.eleves.ens.fr:cof-geek/gestioCOF into qwann/k-fet/stats5ee188e4
- Merge branch 'qwann/k-fet/stats' of git.eleves.ens.fr:cof-geek/gestioCOF into qwann/k-fet/statsAdded 1 commit:
2f1fe15b
- fixing decimal_issue mergingAdded 2 commits:
0ac1eaf4
- 1 commit from branchk-fet
df824a3a
- Merge branch 'k-fet' into qwann/k-fet/statsmentioned in merge request !164
Added ~26 ~25 labels
Hum, l'univers K-Fêt reboot tous les jours entre 6h et 8h du matin. Pas vraiment à minuit...
Si tu fais tes statistiques à partir de minuit, tu vas avoir une partie de la fin de soirée précédente.
Dans l'absolu, ce serait idéal de pouvoir utiliser le futur indicateur d'ouverture de la K-Fêt. En attendant, utiliser 7h c'est une bonne solution.
Pour les ventes d'un article et le nombre d'articles achetés sur un compte, les valeurs sont en principe que entières.
Ça se fait facilement de mettre l'échelle pour n'afficher que des valeurs entières ?
Quand on change de stats sur un article, la page est remontée sans raison (pour l'utilisateur en tout cas :-P )
Sur les graphes de balances d'un compte, des valeurs sont données avant la première commande du compte
un espace de trop
Les fichiers suivants sont ajoutés :
kfet/static/kfet/js/Chart.bundle.js
kfet/static/kfet/js/Chart.bundle.min.js
kfet/static/kfet/js/Chart.js
J'ai l'impression que c'est le même fichier à chaque fois et dans les templates, tu ne fais que appel à:
kfet/static/kfet/js/Chart.bundle.js
.Si une manière standard d'utiliser le plugin est indiquée dans la doc du plugin, utilisons celle-ci (je veux dire: bundle ou pas bundle ?) en version minifiée éventuellement.
EDIT:
Chart.js
etChart.bundle...
ne sont pas les mêmes fichiers. En tout cas, seulementChart.bundle.js
a l'air utilisé.Erkan, je t'avais dit enfin ^^
Au début du fichier de préférence pour les script import.
Et pour plus tard: y'a une raison pour laquelle tu utilises
jQuery
plutôt que$
?Parfois il faut faire ça pour des namespaces schmilblick, donc si ça ne marche pas avec
$
, précise que t'utilisesjQuery
pour cette raisonY'aurait moyen d'utiliser le filtre
join
?Idem
Idem
Ici aussi
n'a rien à faire là
</body>
à virer aussiIndentation
des espaces en plus/en moins
Ce fichier est à virer nan ?
Pourquoi pas :
La durée est récupérée en paramètre (optionnel)
$_GET
(last?duration=...
) et la classe setup la bonne date en fonction du param.Même schmilblick:
stat/balance?duration=...
Même schmilblick :
stat/last?duration=...
En imaginant que le projet s'appelle
gestiocof
, ces 3 classes pourraient aller dansgestiocof/views.py
(actuellement ce seraitcof/views.py
).Discussion à ouvrir sur le nommage du projet/des apps ?
gestiocof
devrait être le nom du projet django contenant les appscof, bda, kfet, bds, ...
ping @mpepin si tu te sens d'ouvrir une issue pour ça
id_prefix = ''
nan ?C'est le bazaar. Le merge des url va te forcer à cleaner ça.
Je repasserai voir après.
👍 ça évite d'avoir 4 urls pour le même truc
Non mais puisque
jQuery
et$
sont des synonymes je ne vois pas le problème :pSera résolu avec la refonte du frontend, ping @champeno
Sera résolu avec la refonte du frontend, ping @champeno
Sera résolu avec la refonte du frontend, ping @champeno
Sera résolu avec la refonte du frontend, ping @champeno
Sera résolu avec la refonte du frontend, ping @champeno
Sera résolu avec la refonte du frontend, ping @champeno
Sera résolu avec la refonte du frontend, ping @champeno
Sera résolu avec la refonte du frontend, ping @champeno
Added 19 commits:
k-fet
9cae3ad0
- Merge branch 'k-fet' into qwann/k-fet/stats1c90608f
- some cleaningAdded 1 commit:
46f343b1
- la K-Fêt se réveille à 7hAdded 1 commit:
0fcb2925
- Frontend tout en JSAdded 1 commit:
ddbcfe0c
- JS commenté et plus propreStatus changed to merged