Permet l'annulation d'un inventaire

lstephan commented

2020-09-11 15:31:11 +02:00

(Migrated from git.eleves.ens.fr)

Lorsqu'on supprime un inventaire, seuls les articles dont c'est le dernier inventaire en date sont affectés.

Fixes #251

Lorsqu'on supprime un inventaire, seuls les articles dont c'est le dernier inventaire en date sont affectés. Fixes #251

mpepin commented

2020-09-17 20:25:18 +02:00

(Migrated from git.eleves.ens.fr)

Lorsqu'on supprime un inventaire, seuls les articles dont c'est le dernier inventaire en date sont affectés.

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 ?

> Lorsqu'on supprime un inventaire, seuls les articles dont c'est le **dernier** inventaire en date sont affectés. 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 ?

lstephan commented

2020-09-18 11:44:34 +02:00

(Migrated from git.eleves.ens.fr)

Je ne pense pas forcément, mais l'overhead pour s'en assurer est négligeable (juste un annotate et un filter en plus), donc je l'ai mis au cas où.

Je ne pense pas forcément, mais l'overhead pour s'en assurer est négligeable (juste un `annotate` et un `filter` en plus), donc je l'ai mis au cas où.

mpepin commented

2020-09-20 23:38:37 +02:00

(Migrated from git.eleves.ens.fr)

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

lstephan commented

2020-09-20 23:58:35 +02:00

(Migrated from git.eleves.ens.fr)

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.

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.

mpepin commented

2020-09-22 21:00:53 +02:00

(Migrated from git.eleves.ens.fr)

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 :

NB : seuls les articles dont c'est le dernier inventaire en date sont affectés

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 : > NB : seuls les articles dont c'est le **dernier** inventaire en date sont affectés

lstephan commented

2020-10-21 15:52:57 +02:00

(Migrated from git.eleves.ens.fr)

added 1 commit

ff7d917a - Message de confirmation plus clair

Compare with previous version

added 1 commit <ul><li>ff7d917a - Message de confirmation plus clair</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/-/merge_requests/457/diffs?diff_id=2209&start_sha=11d591771cb639bd1d93252f9ed7894619b0ee78)

lstephan commented

2020-10-21 15:53:03 +02:00

(Migrated from git.eleves.ens.fr)

resolved all threads

mpepin commented

2020-12-04 17:04:50 +01:00

(Migrated from git.eleves.ens.fr)

added 55 commits

ff7d917a...cc3a4367 - 50 commits from branch master
59dacda3 - Inventory deletion view
f3701d91 - Url and template for InventoryDeleteView
521be6db - Tests
b9699637 - Message de confirmation plus clair
a7cbd2d4 - CHANGELOG

Compare with previous version

added 55 commits <ul><li>ff7d917a...cc3a4367 - 50 commits from branch <code>master</code></li><li>59dacda3 - Inventory deletion view</li><li>f3701d91 - Url and template for InventoryDeleteView</li><li>521be6db - Tests</li><li>b9699637 - Message de confirmation plus clair</li><li>a7cbd2d4 - CHANGELOG</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/-/merge_requests/457/diffs?diff_id=2281&start_sha=ff7d917ae279b2e94790c79982079161d0e2e6e5)

mpepin commented

2020-12-04 17:05:15 +01:00

(Migrated from git.eleves.ens.fr)

enabled an automatic merge when the pipeline for a7cbd2d451 succeeds

enabled an automatic merge when the pipeline for a7cbd2d45165902d304f4e2c3b4f1506cc8674bf succeeds

mpepin commented

2020-12-04 17:13:33 +01:00

(Migrated from git.eleves.ens.fr)

mentioned in commit 72cd55716b

mentioned in commit 72cd55716b14ca3b5adee8e06ab1e2d692ecfe05

Permet l'annulation d'un inventaire #761