-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
6 ajout dun objet prix sur chaque produit #10
base: master
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for matta-nuu ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Super merci pour ces avancées !!
J'ai fait quelques commentaires ;)
Concernant l'utilisation de l'historique des prix, je pense que ça sera utilisé le jour ou on fera des stats du genre "sur l'année dis moi le prix total de chaque produit". Mais ça faut voir avec les utilisateurs quels sont leur besoins réels.
La première utilisation qu'on pourra faire avec les prix c'est calculer le cout d'une recette par personne, et peut etre l'afficher dans la liste des recettes, comme ça tu peux voir quelles sont les plus chères
Top ! Merci pour les fix :) Je pense que tu pourrais maintenant rajouter deux use case simple d'utilisation des prix Ordersrajouter dans les "Order" le prix total de la commande en utilisant les prix? (uniquement si au moins un des produits de la commande à un prix configuré, i.e. si le total est > 0) RecipieCalculer le prix par personne d'une recette. On pourrait afficher ce prix dans la liste des recettes avec une nouvelle colonne? Avec le meme warning si tous les produits n'ont pas de prix |
Je tente la PR en mode draft.
J'ai normalement corrigé l'erreur que tu avais Sebastian, ainsi que tes suggestions.
Dis-moi si c'est mieux, et ce que l'on fait avec les dates dans l'historique des prix.