Rend le code de BdA-Revente plus robuste #418
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
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: DGNum/gestioCOF#418
Loading…
Reference in a new issue
No description provided.
Delete branch "Aufinal/clean_code"
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?
shotgun
devient un champ, et nécessite quetirage_done
vailleTrue
(plus de places au shotgun avant que le tirage au sort ne soit fait)Selon moi, ça règle le 3e point de #101.
Il faut une docstring qui indique ce que renvoie cette fonction. C'est obscur sinon.
C'est vrai qu'elle est mal nommée...
Added 20 commits:
master
a63269a4
- more coherent namesb15dcba8
- mail formattingfa3ddd09
- Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/clean_code40f3cf60
- code mortAdded ~27 label
mentioned in merge request !122
Added 18 commits:
master
a9f25374
- Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/clean_codePourquoi as-tu besoin de remplir ce formulaire ?
Idem, pourquoi remplir ce formulaire ?
Idem pour ces deux formulaires…
Bon visiblement mes commentaires précédents n'apparaissent pas dans le code… Je parle des formulaires que tu replis dans la vue
revente
alors qu'ils ne sont pas utilisés, par exemple ligne 274 :Pourquoi
annulform
?Ah j'ai compris my bad
Je pense que les migrations devraient set l'attribut
shotgun
à la bonne valeur (au sens de l'ancienne méthode de calcul), sinon, tous les spectacles au shotgun en prod vont repasser en mode tirage au sortJe viens d'ouvrir !135 qui vise cette branche et améliorerait encore un peu le code à mon sens. Je te laisse regarder et me dire ce que tu en penses.
Une fois que ça et l'histoire des migrations seront réglés, je suis ok pour merge
Added 5 commits:
master
257e5386
- Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/clean_code38dd220c
- modernize reventes_shotgunAdded 1 commit:
07f92097
- change soldTo functioningAdded 2 commits:
9bbc9e93
- change soldTo functioning82b6cea0
- Merge branch 'Aufinal/clean_code' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/clean_codeAdded 2 commits:
e8be072e
- message si inscription a une revente en cours2f70dbf9
- default shotgun for existing rowsAdded 4 commits:
330b45fb
- Style + commentaires28df0355
- Améliore l'algo d'inscription à une revente847345bb
- Merge branch 'Aufinal/clean_code' into Kerl/clean_code_amendacfcddce
- Merge branch 'Kerl/clean_code_amend' into 'Aufinal/clean_code'Status changed to merged
mentioned in commit
6e5c3c8c33