DGNum/gestioCOF
13
2
Fork 2
Code Issues 82 Pull requests 20 Projects Releases 12 Packages Wiki Activity Actions

Rend le code de BdA-Revente plus robuste #418

Merged
lstephan merged 0 commits from Aufinal/clean_code into master 2016-12-21 17:17:49 +01:00
Conversation 20 Commits - Files changed -
lstephan commented 2016-11-13 02:49:02 +01:00 (Migrated from git.eleves.ens.fr)
Copy link
  • shotgun devient un champ, et nécessite que tirage_done vaille True (plus de places au shotgun avant que le tirage au sort ne soit fait)
  • suppression de code mort
  • correction d'un bug sur les reventes de reventes
  • améliorations diverses : commentaires, messages d'erreur
    Selon moi, ça règle le 3e point de #101.
- `shotgun` devient un champ, et nécessite que `tirage_done` vaille `True` (plus de places au shotgun avant que le tirage au sort ne soit fait) - suppression de code mort - correction d'un bug sur les reventes de reventes - améliorations diverses : commentaires, messages d'erreur Selon moi, ça règle le 3e point de #101.
mpepin commented 2016-11-14 07:38:13 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Il faut une docstring qui indique ce que renvoie cette fonction. C'est obscur sinon.

Il faut une docstring qui indique ce que renvoie cette fonction. C'est obscur sinon.
lstephan commented 2016-11-14 13:41:19 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

C'est vrai qu'elle est mal nommée...

C'est vrai qu'elle est mal nommée...
lstephan commented 2016-11-14 15:59:24 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 20 commits:

  • 790e73d7...dc1ed332 - 16 commits from branch master
  • a63269a4 - more coherent names
  • b15dcba8 - mail formatting
  • fa3ddd09 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/clean_code
  • 40f3cf60 - code mort
Added 20 commits: * 790e73d7...dc1ed332 - 16 commits from branch `master` * a63269a4 - more coherent names * b15dcba8 - mail formatting * fa3ddd09 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/clean_code * 40f3cf60 - code mort
lstephan commented 2016-11-14 17:08:56 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added ~27 label

Added ~27 label
lstephan commented 2016-11-22 23:04:09 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in merge request !122

mentioned in merge request !122
lstephan commented 2016-11-23 04:50:27 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 18 commits:

  • 40f3cf60...2196c7dc - 17 commits from branch master
  • a9f25374 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/clean_code
Added 18 commits: * 40f3cf60...2196c7dc - 17 commits from branch `master` * a9f25374 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/clean_code
mpepin commented 2016-12-20 21:22:19 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Pourquoi as-tu besoin de remplir ce formulaire ?

Pourquoi as-tu besoin de remplir ce formulaire ?
mpepin commented 2016-12-20 21:23:36 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Idem, pourquoi remplir ce formulaire ?

Idem, pourquoi remplir ce formulaire ?
mpepin commented 2016-12-20 21:25:44 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Idem pour ces deux formulaires…

Idem pour ces deux formulaires…
mpepin commented 2016-12-20 21:30:46 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

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 :

if 'resell' in request.POST:
         resellform = ResellForm(participant, request.POST, prefix='resell')
        annulform = AnnulForm(participant, prefix='annul')

Pourquoi annulform ?

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 : if 'resell' in request.POST: resellform = ResellForm(participant, request.POST, prefix='resell') annulform = AnnulForm(participant, prefix='annul') Pourquoi `annulform` ?
mpepin commented 2016-12-20 21:45:56 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Ah j'ai compris my bad

Ah j'ai compris my bad
mpepin commented 2016-12-20 22:23:03 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

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 sort

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 sort
mpepin commented 2016-12-21 02:28:01 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Je 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

Je 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
lstephan commented 2016-12-21 03:35:25 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 5 commits:

  • a9f25374...2d1a9d8e - 3 commits from branch master
  • 257e5386 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/clean_code
  • 38dd220c - modernize reventes_shotgun
Added 5 commits: * a9f25374...2d1a9d8e - 3 commits from branch `master` * 257e5386 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/clean_code * 38dd220c - modernize reventes_shotgun
lstephan commented 2016-12-21 03:45:42 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * 07f92097 - change soldTo functioning
lstephan commented 2016-12-21 03:48:12 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 2 commits:

  • 9bbc9e93 - change soldTo functioning
  • 82b6cea0 - Merge branch 'Aufinal/clean_code' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/clean_code
Added 2 commits: * 9bbc9e93 - change soldTo functioning * 82b6cea0 - Merge branch 'Aufinal/clean_code' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/clean_code
lstephan commented 2016-12-21 04:05:15 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 2 commits:

  • e8be072e - message si inscription a une revente en cours
  • 2f70dbf9 - default shotgun for existing rows
Added 2 commits: * e8be072e - message si inscription a une revente en cours * 2f70dbf9 - default shotgun for existing rows
lstephan commented 2016-12-21 14:13:32 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 4 commits:

  • 330b45fb - Style + commentaires
  • 28df0355 - Améliore l'algo d'inscription à une revente
  • 847345bb - Merge branch 'Aufinal/clean_code' into Kerl/clean_code_amend
  • acfcddce - Merge branch 'Kerl/clean_code_amend' into 'Aufinal/clean_code'
Added 4 commits: * 330b45fb - Style + commentaires * 28df0355 - Améliore l'algo d'inscription à une revente * 847345bb - Merge branch 'Aufinal/clean_code' into Kerl/clean_code_amend * acfcddce - Merge branch 'Kerl/clean_code_amend' into 'Aufinal/clean_code'
mpepin commented 2016-12-21 17:17:40 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Status changed to merged

Status changed to merged
mpepin commented 2016-12-21 17:17:49 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in commit 6e5c3c8c33

mentioned in commit 6e5c3c8c335ee4fd523cda07b20af5d306f8e01f
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
devtype -- backend

Au programme : du Python, du Django...

  
devtype -- docs

   
devtype -- frontend

   
devtype -- user interface

Au programme : templating, JS, CSS...

  
difficulty -- easy

   
difficulty -- hard

Difficile et/ou risqué

  
difficulty -- normal

   
Doing

   
domain -- bda

Concerne l'application BdA

  
domain -- bds

   
domain -- cof

Concerne l'application COF

  
domain -- core

Dépendances externes, CI, linting...

  
domain -- kfet

Concerne l'application K-Fêt

  
Good first issue

   
priority -- high

Urgent quoi

  
priority -- low

   
priority -- medium

   
priority -- staff-wanted

L'équipe dirigeante du domaine en question souhaite cette fonctionnalité.

  
status -- development

En cours de développement

  
status -- discussion

Avant de coder, c'est bien de discuter

  
status -- need review

En attente de code review

  
status -- production

L'ajout/la modification est présente en production

  
status -- ready to merge

Plus qu'à merger !

  
status -- todo

Le développement devrait bientôt commencer

  
To Do

   
type -- bug

Signalement ou correction de bug

  
type -- hygiene

Tout ce qui amène à du code plus propre et un repo mieux organisé.

  
type -- improvement

Ajout/Demande d'évolution mineure d'une fonctionnalité existante

  
type -- new feature

Ajout/Demande d'une nouvelle fonctionnalité

  
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

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
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: DGNum/gestioCOF#418
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?