DGNum/gestioCOF
13
2
Fork 1
Code Issues 77 Pull requests 14 Packages Projects Releases 12 Wiki Activity

Fix pour BdA-Revente #567

Merged
lstephan merged 0 commits from Aufinal/bda_fixes into master 2018-04-07 14:55:35 +02:00
Conversation 36 Commits - Files changed -
lstephan commented 2017-10-23 20:59:42 +02:00 (Migrated from git.eleves.ens.fr)
Copy link
  • Cleanup général du code : fichiers de templates, urls, noms de vues...
  • Possibilité de se désinscrire d'un tirage et de s'inscrire aux tirages en cours
  • Commande manage_reventes plus robuste : résiste mieux si gestioCOF est down, utilisation d'attributs de classe/propriétés, log plus précis.
  • Quelques changements de constantes : 5 jours de marge avant un spectacle, et entre 24 et 72h d'attente avant un tirage.
- Cleanup général du code : fichiers de templates, urls, noms de vues... - Possibilité de se désinscrire d'un tirage et de s'inscrire aux tirages en cours - Commande `manage_reventes` plus robuste : résiste mieux si gestioCOF est down, utilisation d'attributs de classe/propriétés, log plus précis. - Quelques changements de constantes : 5 jours de marge avant un spectacle, et entre 24 et 72h d'attente avant un tirage.
mpepin commented 2017-10-25 19:03:51 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Je pense qu'on ne veut pas un RemoveField mais plutôt un RenameField ici

Je pense qu'on ne veut pas un `RemoveField` mais plutôt un [`RenameField`](https://docs.djangoproject.com/en/1.11/ref/migration-operations/#renamefield) ici
mpepin commented 2017-10-25 19:03:54 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Je pense qu'on ne veut pas un RemoveField mais plutôt un RenameField ici

Je pense qu'on ne veut pas un `RemoveField` mais plutôt un [`RenameField`](https://docs.djangoproject.com/en/1.11/ref/migration-operations/#renamefield) ici
mpepin commented 2017-10-25 19:05:21 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Les deux dernières lignes ne sont jamais exécutées

Les deux dernières lignes ne sont jamais exécutées
lstephan commented 2017-10-25 19:07:25 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ça a été fait automatiquement par `makemigrations, mais effectivement ça peut être judicieux.

Ça a été fait automatiquement par `makemigrations, mais effectivement ça peut être judicieux.
mpepin commented 2017-10-25 19:09:37 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

ou directement

return str(obj.attribution.spectacle)
ou directement ```python return str(obj.attribution.spectacle) ```
mpepin commented 2017-10-25 19:26:42 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

pourquoi supprimer ces lignes ? En plus ça fait râler flake8

pourquoi supprimer ces lignes ? En plus ça fait râler flake8
mpepin commented 2017-10-25 19:29:28 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Probablement peu urgent mais est-ce qu'on ne préviendrait pas les gens par mail lorsqu'une revente est annulée dans cette vue ?

Probablement peu urgent mais est-ce qu'on ne préviendrait pas les gens par mail lorsqu'une revente est annulée dans cette vue ?
mpepin commented 2017-10-25 19:32:58 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Je pense qu'on devrait simplifier cette fonction avec un appel à revente.reset() quitte à remodifier le champ date ensuite.

  • ce timedelta(minutes=65) est pas ultra clair, un commentaire expliquant à quoi il sert ne serait pas de trop
Je pense qu'on devrait simplifier cette fonction avec un appel à `revente.reset()` quitte à remodifier le champ date ensuite. + ce `timedelta(minutes=65)` est pas ultra clair, un commentaire expliquant à quoi il sert ne serait pas de trop
mpepin commented 2017-10-25 19:34:57 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Cette vue sert à gérer ses propres reventes c'est bien ça ? Ça mériterait une ligne de docstring pour dire ça je pense

Cette vue sert à gérer ses propres reventes c'est bien ça ? Ça mériterait une ligne de docstring pour dire ça je pense
mpepin commented 2017-10-25 19:40:03 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ce serait plus élégant avec pluralize je trouve, voir ici par exemple

Ce serait plus élégant avec `pluralize` je trouve, voir [ici](https://git.eleves.ens.fr/cof-geek/gestioCOF/blob/master/bda/admin.py#L279) par exemple
mpepin commented 2017-10-25 19:41:48 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Et

sub = 0
for revente in reventes:
    …
    sub +=1

c'est pas terrible non plus et c'est même plus lent que juste len(reventes).

Et ```python sub = 0 for revente in reventes: … sub +=1 ``` c'est pas terrible non plus et c'est même plus lent que juste `len(reventes)`.
mpepin commented 2017-10-25 19:45:32 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Quelques tests aussi ce serait pas mal, t'as la motiv de le faire ou pas ?

Quelques tests aussi ce serait pas mal, t'as la motiv de le faire ou pas ?
lstephan commented 2017-10-25 19:46:10 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Yup, quand j'aurai de quoi coder sous la main =)

Yup, quand j'aurai de quoi coder sous la main =)
mpepin commented 2017-10-25 19:46:51 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

nice :)

nice :)
lstephan commented 2017-10-26 12:23:36 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

À propos, s'il y a des meilleures idées pour les noms de liens dans home.html, je suis preneur.

À propos, s'il y a des meilleures idées pour les noms de liens dans `home.html`, je suis preneur.
lstephan commented 2017-10-26 12:40:22 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/263/diffs?diff_id=699&start_sha=6a6549e0d72937d9f5adaf7bf43ff090150b4891#7653e3fd1cd134046091cb5b75b07fc781da5d31_14_14)
lstephan commented 2017-10-26 12:40:22 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/263/diffs?diff_id=699&start_sha=6a6549e0d72937d9f5adaf7bf43ff090150b4891#7653e3fd1cd134046091cb5b75b07fc781da5d31_14_14)
lstephan commented 2017-10-26 12:40:22 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/263/diffs?diff_id=699&start_sha=6a6549e0d72937d9f5adaf7bf43ff090150b4891#7f6176014993eb92d98ac833e4477fb8553b642f_50_51)
lstephan commented 2017-10-26 12:40:22 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/263/diffs?diff_id=699&start_sha=6a6549e0d72937d9f5adaf7bf43ff090150b4891#7f6176014993eb92d98ac833e4477fb8553b642f_88_91)
lstephan commented 2017-10-26 12:40:22 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/263/diffs?diff_id=699&start_sha=6a6549e0d72937d9f5adaf7bf43ff090150b4891#40b8549eb07ad8686985e89d4891f3b4346d8e85_424_418)
lstephan commented 2017-10-26 12:40:22 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/263/diffs?diff_id=699&start_sha=6a6549e0d72937d9f5adaf7bf43ff090150b4891#40b8549eb07ad8686985e89d4891f3b4346d8e85_460_453)
lstephan commented 2017-10-26 12:40:22 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * 785555c0 - Misc fixes [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/263/diffs?diff_id=699&start_sha=6a6549e0d72937d9f5adaf7bf43ff090150b4891)
mpepin commented 2017-10-30 13:08:08 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Je chipote mais ces variables définies dans les deux branches du if qu'on peut miraculeusement utiliser après le if je trouve que c'est vraiment une des horreurs de python. On ne pourrait pas définir winner = None avant le if plutôt que dans le else ?

Je chipote mais ces variables définies dans les deux branches du `if` qu'on peut miraculeusement utiliser après le `if` je trouve que c'est vraiment une des horreurs de python. On ne pourrait pas définir `winner = None` avant le `if` plutôt que dans le `else` ?
mpepin commented 2017-11-01 11:26:12 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

added 46 commits

  • 785555c0...273e6374 - 45 commits from branch master
  • e1794a65 - Merge branch 'master' into Aufinal/bda_fixes

Compare with previous version

added 46 commits * 785555c0...273e6374 - 45 commits from branch `master` * e1794a65 - Merge branch 'master' into Aufinal/bda_fixes [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/263/diffs?diff_id=708&start_sha=785555c05cc874dfc2a9542608c0e94baffccc2e)
mpepin commented 2017-11-01 11:28:55 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

created #180 to continue this discussion

created #180 to continue this discussion
mpepin commented 2017-11-01 11:28:55 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in issue #180

mentioned in issue #180
lstephan commented 2017-12-19 11:53:55 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

added 3 commits

  • f18959c0 - BdA-Revente: meaningful names, some help tests
  • 91bdf118 - Coding style: python's scope sucks
  • 5f9f222c - Merge branch 'Kerl/bda_fixes' into 'Aufinal/bda_fixes'

Compare with previous version

added 3 commits * f18959c0 - BdA-Revente: meaningful names, some help tests * 91bdf118 - Coding style: python's scope sucks * 5f9f222c - Merge branch 'Kerl/bda_fixes' into 'Aufinal/bda_fixes' [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/263/diffs?diff_id=722&start_sha=e1794a654fa7b0e70130ffa8034eff54c0cd7527)
lstephan commented 2017-12-19 12:56:23 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

added 5 commits

  • 21252801 - Add some tests
  • dfa8c1a1 - Merge branch 'Aufinal/bda_fixes' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/bda_fixes
  • f1bbade0 - Better labels for revente objects
  • 1783196a - Management view only deals with Revente objects
  • 9a877397 - Use new method in admin

Compare with previous version

added 5 commits * 21252801 - Add some tests * dfa8c1a1 - Merge branch 'Aufinal/bda_fixes' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/bda_fixes * f1bbade0 - Better labels for `revente` objects * 1783196a - Management view only deals with Revente objects * 9a877397 - Use new method in admin [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/263/diffs?diff_id=723&start_sha=5f9f222cda6c8c79a49541e05d239e5d02de33ef)
mpepin commented 2018-01-10 17:35:41 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

  • 91119f68 - Ne pas oublier avant de passer en prod…

Compare with previous version

added 1 commit * 91119f68 - Ne pas oublier avant de passer en prod… [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/263/diffs?diff_id=734&start_sha=9a8773978c52defaf98dd7fd58babd4961273dd5)
lstephan commented 2018-01-10 20:00:52 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

added 9 commits

  • 91119f68...44eee9be - 8 commits from branch master
  • 501d592d - Merge branch 'master' into Aufinal/bda_fixes

Compare with previous version

added 9 commits * 91119f68...44eee9be - 8 commits from branch `master` * 501d592d - Merge branch 'master' into Aufinal/bda_fixes [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/263/diffs?diff_id=739&start_sha=91119f68bc5deef627baa47ec7fb80e56c0bb514)
lstephan commented 2018-01-10 20:14:45 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * 52fd4961 - Fix model test [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/263/diffs?diff_id=741&start_sha=501d592d2f40d6b680e7376e40a37ca0d445d5b9)
mpepin commented 2018-01-10 20:16:15 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

resolved all discussions

resolved all discussions
mpepin commented 2018-04-07 14:51:33 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 63 commits

  • 52fd4961...568a58c5 - 62 commits from branch master
  • 87a67221 - Merge branch 'master' into Aufinal/bda_fixes

Compare with previous version

added 63 commits * 52fd4961...568a58c5 - 62 commits from branch `master` * 87a67221 - Merge branch 'master' into Aufinal/bda_fixes [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/263/diffs?diff_id=817&start_sha=52fd49616dab6515ab6e113ccb405878c59f18dd)
mpepin commented 2018-04-07 14:51:56 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

enabled an automatic merge when the pipeline for 87a6722143 succeeds

enabled an automatic merge when the pipeline for 87a672214354c08bdc8142f8ec8d323cc098e342 succeeds
mpepin commented 2018-04-07 14:55:36 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

merged

merged
mpepin commented 2018-04-07 14:55:36 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in commit 25f4c64835

mentioned in commit 25f4c648357e307893c14db1161cd1cf00c30153
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é

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
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#567
No description provided.
Delete branch "Aufinal/bda_fixes"

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?