added 2 commits

• 333a3529 - presta add and delete products on sale
• ebe13f85 - Products families admin with conditionnal display on tav env

Compare with previous version

added 10 commits

• ebe13f85...1ac5109e - 9 commits from branch ssa-gironde
• f94a0edd - fix merge conflicts

Compare with previous version

unmarked as a Work In Progress

added 1 commit

• f5ec1cab - add doc on new env param

Compare with previous version

Le menu des familles de produits est toujours accessible par un admin lorsque PRESTA_EXTRA_DATA est à 0. Est-ce que ça te parait pertinent et pas trop galère de conditionner cet affichage à cette variable d'environnement ?

Si un prestataire ajoute deux fois la même catégorie de produits (ça risque d'arriver), en prod il va se retrouver sur une page du type "Ooops" car l'exception n'est pas gérée. Ce serait bien, soit de l'empêcher d'ajouter une famille déjà sélectionnée (ce qui a ma préférence car cela aurait aussi l'avantage de préselectionner sur la famille suivante à chaque clic sur le bouton + et donc d'économiser jusqu'à une vingtaine de clics), soit, si c'est galère, d'afficher un message lui expliquant qu'il ne peut pas ajouter deux fois la même famille. Qu'en penses-tu ?

ce qui est lié aux familles de produits devrait être conditionné à presta_extra_data et non à tav_env non ?

J'ai voulu faire ça pour minimiser le nombre de variables d'environnement qu'on injecte dans le code, et donc gagner du temps et rendre le code plus lisible. Je me suis dit que si les autres utilisateurs TAV voient un nouveau menu dans l'admin dont ils ne se servent pas, cela n'aura pas spécialement d'impact.

Je me trompe peut-être, si tu penses que ça vaut le coup de changer de paramètre je le fais !

Je suis favorable à utiliser le moins possible les variables d'environnement dans le code (et j'aurais d'ailleurs aimé aller plus loin que ça en ne paramétrant que ce qui poserait réellement problème à Montpellier, mais ça c'est une autre histoire dont nous reparlerons peut-être, ou pas).

je suis d'accord

resolved all discussions

Si un prestataire ajoute deux fois la même catégorie de produits (ça risque d'arriver), ...

Bien vu, j'essaye ça

@Yvon je viens de penser à un cas d'erreur que ça pourrait engendrer : mettons que le prestataire veuille ajouter 3 familles de produits. Il clique 3 fois sur le + à la suite pour ensuite remplir les champs.

Si on enlève les options déjà sélectionnées au fur et à mesure, on n'aura pas les bonnes options enlevées du select.

Pour que ça fonctionne il faudrait que tous les select soient réactifs entre eux, en temps réel à chaque changement, ce qui est faisable mais demanderait trop de travail à mon sens.

Je pense donc plutôt partir sur ta deuxième proposition, afficher un message d'erreur.

corrigé : j'ai ajouté un warning à côté de chaque famille de produit si une autre existe déjà avec la même famille sélectionnée.

• ajout d'un message d'erreur si l'utilisateur valide quand même.

top

added 1 commit

• 9c3e9d1a - prestataire infos: deal with duplicates product family

Compare with previous version

resolved all discussions

merged

mentioned in commit 8be42f4b