Déplace le champ paid des participants aux attributions #665

Merged
lstephan merged 0 commits from Aufinal/paid_attributions into master 2019-06-17 21:59:02 +02:00
lstephan commented 2019-06-08 16:07:49 +02:00 (Migrated from git.eleves.ens.fr)

Sur demande du BdA les champs paid et paymenttype sont déplacés sur les attributions au lieu des participants. Du coup:

  • on crée un Manager custom pour agréger les paiements au niveau des participants
  • on réécrit 2-3 vues de façon plus moderne pour accomoder les changements
  • l'interface admin est légèrement modifiée
Sur demande du BdA les champs `paid` et `paymenttype` sont déplacés sur les attributions au lieu des participants. Du coup: * on crée un `Manager` custom pour agréger les paiements au niveau des participants * on réécrit 2-3 vues de façon plus moderne pour accomoder les changements * l'interface admin est légèrement modifiée
bclement commented 2019-06-17 20:50:19 +02:00 (Migrated from git.eleves.ens.fr)

Useless local variable?

Useless local variable?
bclement commented 2019-06-17 20:54:10 +02:00 (Migrated from git.eleves.ens.fr)

pass ne sert à rien ici puisqu'il y a d'autres statements dans le block

`pass` ne sert à rien ici puisqu'il y a d'autres statements dans le block
bclement commented 2019-06-17 20:57:31 +02:00 (Migrated from git.eleves.ens.fr)

Worth it to have a short comment explaining what is going on here

Worth it to have a short comment explaining what is going on here
bclement commented 2019-06-17 21:06:02 +02:00 (Migrated from git.eleves.ens.fr)

Pendant qu'on y est ça serait cool de faire un warning pour les admins ici :)
(Juste une idée comme ça)

Pendant qu'on y est ça serait cool de faire un warning pour les admins ici :) (Juste une idée comme ça)
lstephan commented 2019-06-17 21:38:56 +02:00 (Migrated from git.eleves.ens.fr)

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/361/diffs?diff_id=1201&start_sha=b11e35616c297e37ca4ff8afa39fbe39449337a4#c45908b588ee57f940e5576626d8e5af3edd9cff_120_120)
lstephan commented 2019-06-17 21:38:57 +02:00 (Migrated from git.eleves.ens.fr)

added 5 commits

Compare with previous version

added 5 commits <ul><li>ba5aa6da - Remove useless local variable</li><li>20bb9fe5 - Remove debug log</li><li>edd92bea - Add logging call</li><li>4f15b820 - Use manager from queryset</li><li>46e73059 - Meilleur décorateur</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/361/diffs?diff_id=1201&start_sha=b11e35616c297e37ca4ff8afa39fbe39449337a4)
lstephan commented 2019-06-17 21:40:38 +02:00 (Migrated from git.eleves.ens.fr)

added 1 commit

Compare with previous version

added 1 commit <ul><li>d7d0daea - Commentaire dans la fonction</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/361/diffs?diff_id=1202&start_sha=46e7305953231faea2c79c88a005e31a83c18d54)
lstephan commented 2019-06-17 21:42:20 +02:00 (Migrated from git.eleves.ens.fr)

resolved all discussions

resolved all discussions
lstephan commented 2019-06-17 21:59:02 +02:00 (Migrated from git.eleves.ens.fr)

merged

merged
lstephan commented 2019-06-17 21:59:02 +02:00 (Migrated from git.eleves.ens.fr)

mentioned in commit 4598abc721

mentioned in commit 4598abc7210b5b7ab9c73eb8ee8b71214840e812
Sign in to join this conversation.
No reviewers
No milestone
No project
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#665
No description provided.