Kerl/clean code amend #439
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#439
Loading…
Reference in a new issue
No description provided.
Delete branch "Kerl/clean_code_amend"
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?
Ajoute quelques commentaires
Reformate le texte en 80 colonnes
Améliore l'algo d'inscription à la revente en inscrivant les gens à la revente ayant le moins d'inscrits
mentioned in merge request !114
Dans l'ancienne version du code, on essaie d'inscrire les gens à une revente disponible (le
else
) même si des places au shotgun existaient. Àl'époque il était théoriquement impossible d'avoir les deux en même temps mais désormais c'est possible, il m'a donc semblé légitime de séparer complètement les deux cas.
Je le signale pour que cette différence n'échappe pas à la review, confirme moi que c'est correct :)
Nope, dans l'ancienne version du code il était tout aussi possible d'avoir des places au shotgun + une revente en cours ; et apparemment j'inscrivais le participant a une revente même s'il y avait des places au shotgun.
Heureusement, pour que ça se produise il faudrait une invraisemblable conjonction d'offre et de demande, donc ce n'est jamais arrivé ; mais c'était une erreur =)
Ca manque d'un message pour le prévenir quand même qu'il a été inscrit ; mais je vais faire ça sur la branche principale.
T'es sûr ? Il y a déjà le message de succès en vert en haut de l'écran
Il te dit uniquement que tu t'es inscrit pour recevoir les mails des prochaines reventes, pas que tu t'es fait inscrire à une revente en cours, si ?
Hum, right
Il va falloir rebase, mais je suis OK pour merge.
Added 11 commits:
Aufinal/clean_code
847345bb
- Merge branch 'Aufinal/clean_code' into Kerl/clean_code_amendStatus changed to merged
mentioned in commit
acfcddce07