Sc setup stock app
-
18 18 location_dest_id = settings.MEALS_LOC_ID 19 19 elif movement_type == 'autoconso': 20 20 location_dest_id = settings.AUTOCONSO_LOC_ID 21 elif movement_type == 'caisse-invendus-supercafoutch': -
Developer
D'autres coops pourraient aussi avoir une "caisse invendus", donc j'enlèverais le nom "supercafoutch" de la variable.
De manière générale, je pense qu'il vaut mieux éviter de personnaliser du code à ce point pour cette raison ! C'est ce qu'on s'était dit avec François.
-
-
-
32 32 <option value="losses">Pertes</option> 33 33 <option value="meals">Repas salariés</option> 34 34 <option value="autoconso">Autoconsomation</option> 35 <option value="caisse-invendus-supercafoutch">Caisse invendus (Supercafoutch)</option> -
Developer
Plutôt que de mettre
(Supercafoutch)
, il vaut mieux paramétrer l'affichage de cette section en fonction d'un paramètre d'environnement. C'est fait assez régulièrement dans le code et c'est de cette façon qu'on personnalise le code par foodcoop. -
-
-
28 31 ['date_done', "<=", date_to], 29 32 ['state', "=", 'done']] 30 33 34 if movement_type == 'caisse-invendus-supercafoutch': 35 c.append(['picking_type_id', "=", picking_type_id]) -
Developer
comment se fait-il que tu doive rajouter un
picking_type_id
dans ce cas précis ? Dans les autres cas, préciser lelocation_dest_id
suffit. -
-
-
151 151 ALLOW_FOUR_DIGITS_IN_RECEPTION_PRICE = True 152 152 153 153 REMOVE_15_MINUTES_AT_SHIFT_END = False 154 155 LOSSES_LOC_ID = 33 156 LOSSES_PICKING_TYPE_ID = 10 157 AUTOCONSO_LOC_ID = 27 158 AUTOCONSO_PICKING_TYPE_ID = 7 159 160 CUSTOMERS_LOC_ID = 9 -
Developer
Je mettrais un nom plus précis, qui correspond plus à la "caisse invendus" (sauf si tu as eu une bonne raison de le nommer comme ça !)
-
-
-
Developer
Je ne connais pas la fonctionnalité, mais c'est normal que tu rajoutes la possibilité de récupérer ces mouvements de stocks mais pas de les créer dans le module ?
-
Developer
Avec la modification que je propose ce ne sera plus nécessaire, mais pour la suite, si un paramètre est nécessaire (dans le sens où quelque chose bug si le paramètre est absent), il faut le rajouter dans tous les fichiers d'exemple de config des coops.
Après si on paramètre l'affichage on ne devrait pas pouvoir cliquer sur quelque chose qui n'est pas pour nous, mais je préfère préciser car c'est le cas avec ta branche actuellement !
-
added 5 commits
-
563e19fa...273c7bbc - 4 commits from branch
dev_cooperatic
- 85afcfb0 - Merge branch 'dev_cooperatic' into sc-setup-stock-app
Toggle commit list -
563e19fa...273c7bbc - 4 commits from branch
-
merged
Toggle commit list