Hum, en fait j'ai ouvert une nouvelle issue ( #148 désolé @mpepin) pour qu'on réfléchisse à ce qu'on veut comme cases dans ce truc (un prix individuel, ou le prix total de l'item ? etc.) et comment rentrer ça dans la BDD...

delobell commented

2017-02-13 09:44:50 +01:00

(Migrated from git.eleves.ens.fr)

Oww ça fait du bien.
table-condensed passe le padding des cellules de 8px à 5px, mais je pense qu'il faut aller encore au-delà est le mettre à 0. Histoire que ça fasse (un peu moins) tableau sans fin.

Oww ça fait du bien. `table-condensed` passe le padding des cellules de 8px à 5px, mais je pense qu'il faut aller encore au-delà est le mettre à 0. Histoire que ça fasse (un peu moins) tableau sans fin.

mpepin commented

2017-02-13 12:23:19 +01:00

(Migrated from git.eleves.ens.fr)

Très joli :)

mpepin commented

2017-02-13 12:31:21 +01:00

(Migrated from git.eleves.ens.fr)

En revanche deux points

`ModelForm`

Pourquoi ne pas utiliser un ModelForm ?

J'ai l'impression qu'il suffit de surcharger la méthode save à un ModelForm pour avoir le même comportement (e.g. tenir compte de la box_capacity). Ça me semble plus adéquat et le code devient plus simple dans la vue.

`init`

Il y a beaucoup de choses qui me semblent inutiles dans la méthode __init__. En fait tout ce qui peut être obtenu à l'aide de article.* me semble surperflu (typiquement name, category, category__name, etc). Après je mesure peut-être mal à quel point ça simplifie l'affichage dans le template de tout avoir là mais ça devrait au moins être remis en question à mon avis.

En revanche deux points ##### `ModelForm` Pourquoi ne pas utiliser un `ModelForm` ? J'ai l'impression qu'il suffit de surcharger la méthode save à un `ModelForm` pour avoir le même comportement (e.g. tenir compte de la `box_capacity`). Ça me semble plus adéquat et le code devient plus simple dans la vue. ##### `__init__` Il y a beaucoup de choses qui me semblent inutiles dans la méthode `__init__`. En fait tout ce qui peut être obtenu à l'aide de `article.*` me semble surperflu (typiquement `name`, `category`, `category__name`, etc). Après je mesure peut-être mal à quel point ça simplifie l'affichage dans le template de tout avoir là mais ça devrait au moins être remis en question à mon avis.

delobell commented

2017-02-13 13:03:36 +01:00

(Migrated from git.eleves.ens.fr)

Oui y'a du nettoyage possible sur le python.
Ça peut être fait sur cette MR ou sur une autre

Oui y'a du nettoyage possible sur le python. Ça peut être fait sur cette MR ou sur une autre

lstephan commented

2017-02-13 14:34:56 +01:00

(Migrated from git.eleves.ens.fr)

Added 1 commit:

83e41c29 - css tweaks

Added 1 commit: * 83e41c29 - css tweaks

lstephan commented

2017-02-23 12:50:10 +01:00

(Migrated from git.eleves.ens.fr)

Deleted source branch aufinal/command_interface

Deleted source branch `aufinal/command_interface`

lstephan commented

2017-02-23 12:51:20 +01:00

(Migrated from git.eleves.ens.fr)

Restored source branch aufinal/command_interface

Restored source branch `aufinal/command_interface`

lstephan commented

2017-03-06 01:29:38 +01:00

(Migrated from git.eleves.ens.fr)

Added 71 commits:

83e41c29...95b129e3 - 69 commits from branch k-fet
61e2fedb - Merge branch 'k-fet' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/command_interface
8e31fae3 - last tweaks

Added 71 commits: * 83e41c29...95b129e3 - 69 commits from branch `k-fet` * 61e2fedb - Merge branch 'k-fet' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/command_interface * 8e31fae3 - last tweaks

delobell commented

2017-03-06 01:39:34 +01:00

(Migrated from git.eleves.ens.fr)

Enabled an automatic merge when the build for 8e31fae340 succeeds

Enabled an automatic merge when the build for 8e31fae3409568e807b74bef4154a7d08484c581 succeeds

delobell commented

2017-03-06 01:39:40 +01:00

(Migrated from git.eleves.ens.fr)

Canceled the automatic merge

delobell commented

2017-03-06 01:40:43 +01:00

(Migrated from git.eleves.ens.fr)