Interface d'inventaires #504
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#504
Loading…
Reference in a new issue
No description provided.
Delete branch "Aufinal/inventaires"
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?
Effet de bord : authentification plus jolie sur
order_create
etorder_to_inventory
.Fix #141
added 1 commit
eb7d436b
- Add "count finished" checkboxCompare with previous version
added 2 commits
31888e33
- simpler jquery selection998838ca
- Add update buttonCompare with previous version
added 1 commit
0c212383
- Change misc field when updatingCompare with previous version
Vu que le reste de la logique du formulaire (les attributs des inputs...) est dans le template directement, tu veux pas aussi y laisser celle-ci ?
Là ça fait une partie ici et une autre partie ailleurs. Je pense que c'est mieux de tout avoir à un seul endroit.
Je pense que c'est mieux de laisser ça en dehors de la
table
(et donc y aller à coup decol-sm-8
ou whatever, etc).Si tu trouves un moyen pas trop dégueu d'éviter le double input submit aussi...
A la place des
col-md-2
vides, tu peux utiliser qqch du typecol-md-offset-2
sur lescol-md-8
suivants en théorie.Le délire des
+
, c'est pour convertir en nombre ?Wut ?
Et sinon, autant faire de
conflicts
unSet
directement ?Il a dû se perdre
Plutôt que de faire ça, tu dois pouvoir sélectionner le formulaire et lui coller un event listener
submit
.Ça permet de détecter aussi une validation du formulaire au clavier.
On en profite pour avoir ce bout de code à un seul endroit ? (
kfet.js
par exemple)Pour l'instant, juste une petite fonction qui fait exactement ça (en prenant optionnellement l'url de la ws, par défaut celle de k-psul).
Et si on se chauffe à un moment, on peut ajouter un petit indicateur du statut du ws (sur K-Psul en particulier). Comme ça l'utilisateur sait pourquoi l'historique, la caisse, etc, ne se mettent pas à jour quand ça arrive.
En commandant sur K-Psul en parallèle, si je clique sur le bouton Maj qui apparaît, le valeur affichée dans le stock calculé est la même que celle de misc et n'a pas l'air de prendre en compte les caisses en réserve et arrière.
Edit: en fait si ça fonctionne, le schmilblick qui s'est passé est que les box_capacity avaient toutes disparues. Je ne sais pas si c'est à cause de moi en faisant des manips avec la console du navigateur ou si ça vient du code...
Je suis d'accord, c'est dans mes petits projets à court terme =)
Ça marche ;-)
Yup, je peux pas les convertir directement quand j'appelle
.val()
parce que ça ne distingue plus''
de'0'
IIRC j'avais fait ça parce que les
<tr>
n'ont de bordure qu'en haut donc ça faisait moche de pas avoir de bordure en bas de la table. Après je peux toucher au CSS pour faire çaEdit: et accessoirement parce que moi et le centrage vertical en CSS ça fait deux...
added 4 commits
e54324e9
- Use col-offset50cb6e51
- Use Set() natively6ac1241b
- Listen to submit instead530aafad
- Use widget_tweaksCompare with previous version
Done ; juste pour info, ce listener là prend aussi en compte la validation avec
Enter
Normalement, box_capacity n'est pas touché par le WS donc je ne vois pas pourquoi ça les modifierait.
added 2 commits
063446ef
- Use columns for authentication and submit8da832c1
- Use nice authentication in orders tooCompare with previous version
added ~15 label
Quand il y a un conflit, même après résolution, je n'arrive pas à valider le nouvel inventaire, le message qui apparaît:
Conflits possibles :
1664 blanc ()
added 1 commit
e20ab2f3
- Use set delete()Compare with previous version
@manet si tu peux jeter un oeil pour voir si ça correspond à ce que tu voulais ça peut être nice =)
Les boutons MàJ apparaissent dès le chargement de la page de création d'inventaires, et ne s'affichent plus malgré le fait que
Compte terminé
soit coché et qu'il y ait eu des commandes sur K-Psul.En gros, y'a un problème.
Par ailleurs, j'attends de savoir ce qu'en dit @manet avant de merge
@lstephan niveau apparence c'est très bien, et le bouton "stock terminé", s'il fonctionne comme j'imagine c'est génial.
Par contre chez moi ça marche pas (j'ai peut-être pas une BDD en bon état, là j'essaye de me recréer ça de zéro) ; je te tiens au courant si j'arrive à le faire tourner, et sinon ce qui ne va pas plus précisément.
Par contre, si on pouvait faire des demi-caisses (entrer "0,5" dans les cases des caisses) ça serait cool. Ça marche déjà pour le JS (il m'affiche le bon montant dans la colonne "stock total"), mais pas à la validation du formulaire. :)
Mais sinon, wow. :)
@manet : un petit script pour gagner du temps : raz.sh ;)
La ligne avec pip me sert parce que je me balade entre des branches en Django 1.11 et des branches en Django 1.8 mais tu peux la commenter si tu veux gagner du temps.
added 2 commits
b113a57b
- Fix update function40da3bc2
- Listen on inputCompare with previous version
@delobell : Normalement le problème est réglé (chez moi) par un Ctrl+Maj+r (ou un restart de
runserver
).Par ailleurs, j'ai revert le fait de listen sur
submit
car ça créait une boucle dans la validation (leconfirm
soumet le formulaire, qui est intercepté par js, et ça recommence).@manet : sur ma version, je n'ai pas de problème à compter des demi-caisses... Accessoirement il y avait effectivement quelques bugs, mais ils devraient être réglés.
Pour la boucle de validation : OK, je crois que c'était mon problème...
@lstephan : y'a pas un vieux bail de renvoyer true ou false pour valider le formulaire et éviter la boucle ?
edit: ah mais avec le dialog ça devient relou, bon bah soit alors
@manet : même chose pour les demi-caisses. Ça fonctionne du moment que tu te retrouves avec un entier à la fin, sinon ça indique une erreur. Seems legit.
Si tu me confirmes que c'est bon pour toi, on est parti.
[Youpi, ça remarche chez moi]
Niveau comportement, je pense qu'il serait mieux de n'enregistrer dans l'inventaire que ceux qui sont cochés, non ?
Sinon, chez moi le fait de décocher la case a le même comportement que cliquer sur le bouton "MàJ" puis décocher la case. Je trouve que c'est pas une bonne idée, il faudrait qu'on puisse juste décocher et recocher la case sans que ça modifie les informations qu'on avait entrées...
@delobell Pour les demi-caisses : en effet, ça marche. C'est juste un helptext chez moi qui me demandait de choisir un nombre entre 0 et 1...
Mis à part les deux points plus haut, c'est très bien pour moi, on peut y aller.
added 80 commits
master
f46ba0dd
- Merge branch 'master' into Aufinal/inventaires32474a68
- Don't update input when unchecking51acb4e0
- Use new WS classCompare with previous version
resolved all discussions
mentioned in commit
0405eee7a7
merged