Stats K-Fêt - Python side #512
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#512
Loading…
Reference in a new issue
No description provided.
Delete branch "aureplop/fix_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?
K-Fêt
teamkfet
K-Fêt - Stats
kfet.<...>.stat.<type>.list
: manifest of proposed stats to userkfet.<...>.stat.<type>
: return stat datasets according to GET params (scale, begin, end, ...)Scale
begin
,end
,last
...begin, end
intervals (chunks)DayScale, WeekScale, MonthScale
KFET_WAKES_UP_AT
K-Fêt - Misc
statistic.js
account_read_title.html
views.py
PkUrlMixin
: use withSingleObject...
class-based view, it gets object by filtering model objects onpk_url_kwarg
added 1 commit
b3a9ad8a
- clean mixinCompare with previous version
Est ce que c'est vraiment utile de le définir comme ça si tu override son fonctionnement dans la plupart des classes ? Pour moi, définir directement
get_chunk_start
est plus lisible et intuitif que définiroffset_to_chunk_start
.added 1 commit
1bb83ccd
- simplify StatScaleCompare with previous version
Un petit oubli =)
Du coup avec ça on perd la responsivité de la taille ou non ?
Ça ne vient pas de toi mais les commentaires sont un peu déroutants ^^
Je vais corriger ça
Je viens de voir que ça pose problème d'enlever ça (enfin son équivalent avec begin et end) quand il n'y a aucune opération sur le compte
Actuellement, quand un compte est créé dans la dernière année, tout est tronqué à sa date de création ; est ce que c'est volontaire, et si oui pourquoi ne pas mettre des zéros avant ?
added 2 commits
769c3763
- delete debug msg87bc90ec
- Begin/end balance stat graphCompare with previous version
@lstephan
Tout le temps
prend en compte la date de création du compte mais pas les autres. Ça m'a l'air déjà plus legit comme çaÇa garde le ratio width/height pour le moment, donc sur une tablette (où tout n'est pas encore sur une colonne), ça fait effectivement petit.
Je pars retrouver l'option de chart.js pour contrôler ça...
Ok c'est juste que j'avais des opérations qui dataient d'avant la création du compte, du coup ça faisait des trucs étranges.
added 1 commit
903da795
- clean kfet statistic.jsCompare with previous version
Ça sert dans les stats ou c'est juste par principe ?
Ok apparemment c'est censé être dans les labels, mais je les ai pas...
added 2 commits
6d2e150a
- clean align10d2b58f
- clean some commentsCompare with previous version
Bonne question!
Oui c'était utilisé dans les labels d'opérations. Sauf que en même-temps ces labels ne sont pas utilisés pour l'affichage du graph...
Donc je l'ai mis là parce que ça peut toujours être utile (et que c'est utilisé dans une vue de stats).
Est-ce que @narmanli pourrait passer par là pour nous expliquer quel était le fond de sa pensée avec ces labels ?
C'est bon pour toi avec les derniers commits ?
added 1 commit
df467767
- fix default GET param 'types' on operations statsCompare with previous version
added 1 commit
7989a07b
- cleaner fixCompare with previous version
added 2 commits
f13d1072
- Add simple tests for the stat viewsb383923d
- Merge branch 'Kerl/test_stats' into aureplop/fix_statsCompare with previous version
added 1 commit
278459e8
- typo.......Compare with previous version
added 2 commits
885e40fd
- cleaner scalesdc07b072
- Flatten scale args in GET params of stats urlsCompare with previous version
changed title from {-Aureplop/fix stats-} to {+Stats K-Fêt - Python side+}
added 1 commit
85ba44c2
- Tests the redirection using the appropriate methodCompare with previous version
merged
mentioned in commit
fe66a6ef6b