Permet l'annulation d'un inventaire #761
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
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: DGNum/gestioCOF#761
Loading…
Reference in a new issue
No description provided.
Delete branch "Aufinal/inventory_delete"
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?
Lorsqu'on supprime un inventaire, seuls les articles dont c'est le dernier inventaire en date sont affectés.
Fixes #251
Je pense que je comprends la logique : s'il y a eu un nouvel inventaire sur un article alors le nouvel inventaire fait foi et on ne doit pas toucher l'article. Right ?
Il y a des use-cases où on veut annuler un inventaire qui n'est pas le dernier ?
Je ne pense pas forcément, mais l'overhead pour s'en assurer est négligeable (juste un
annotate
et unfilter
en plus), donc je l'ai mis au cas où.Le fond de ma question c'était plutôt de savoir si on devrait lever une erreur si quelqu'un essaie d'annuler un vieil inventaire
De mon point de vue, je ne connais pas assez le fonctionnement actuel de la K-Fêt et on peut permettre cette option sans trop de difficulté, donc je ne vois pas pourquoi renvoyer une erreur :slight_smile:
S'il faut, je peux faire une vraie page de confirmation avec les changements de stocks que cela impliquerait.
Détailler tous les changements ça me paraît un peu overkill mais je pense que la popup de confirmation devrait expliquer ce qui se passe, par exemple :
added 1 commit
Compare with previous version
resolved all threads
added 55 commits
master
59dacda3
- Inventory deletion viewf3701d91
- Url and template for InventoryDeleteView521be6db
- Testsb9699637
- Message de confirmation plus claira7cbd2d4
- CHANGELOGCompare with previous version
enabled an automatic merge when the pipeline for
a7cbd2d451
succeedsmentioned in commit
72cd55716b