search

bruecksen / bakeup

http://bakeup.org
Other
10 stars 2 forks source link

Add UOM table and make use of it in product model and all calculations #469

Closed bruecksen closed 1 month ago

bruecksen commented 1 month ago

@madmooose ich hab mal eine UOM Tabelle eingeführt. Das Gewicht eines Produkts kann jetzt mit verschiedenen Einheiten abgespeichert werden. Bei allen berechnungen wird weight_in_base_unit verwendet. Schau gerne mal drüber und teste auf Staging wenn du bock hast :)

madmooose commented 1 month ago

IMG_20240525_160246 In dem Form (oder dahinter fehlt die Einheit)

madmooose commented 1 month ago

@bruecksen maybe we should write a migration script to convert all products >= 1000g into unit kg.

madmooose commented 1 month ago

IMG_20240525_161216 IMG_20240525_161255 Uom is not used in shop view @bruecksen what do you think? Leave it to clever_round or use uom. I prefer the later but it should still use per 100g on the base price and not per gram. [Edit] Just seen that we don't even have per 100g. We should introduce this for products in uom gram IMG_20240525_161714

madmooose commented 1 month ago

Not sure how to choose the right unit here because if we make the recipe in gram it still makes since to have big amounts of dough in kg IMG_20240525_164223

Same here in the production day summary IMG_20240525_165053

bruecksen commented 1 month ago

@madmooose yo ich sehe viele der Punkte. Mein Ziel war es erstmal UOM einzuführen und den rest einfach wieder funktionsfähig zu machen. Wollen wir deine Punkte zurück ins Ticket oder in ein neues Ticket überführen? Dann kann das erstmal live gehen und ich schaue mir die anderen punkte in ruhe an...

madmooose commented 1 month ago

Ich würde alle Bugs im Frontend fixen. Wenn ich mich recht erinnere, ist das nur das produktgewicht auf der produktionstagkachel und im Sortiment. Dann gerne iterativ weiter.

Am 29. Mai 2024 06:46:53 MESZ schrieb "Matthias Brück" @.***>:

@madmooose yo ich sehe viele der Punkte. Mein Ziel war es erstmal UOM einzuführen und den rest einfach wieder funktionsfähig zu machen. Wollen wir deine Punkte zurück ins Ticket oder in ein neues Ticket überführen? Dann kann das erstmal live gehen und ich schaue mir die anderen punkte in ruhe an...

-- Reply to this email directly or view it on GitHub: https://github.com/bruecksen/bakeup/pull/469#issuecomment-2136504353 You are receiving this because you were mentioned.

Message ID: @.***> -- mobil gesendet Niels Göttsch Gut Hemmelmark 5 24360 Barkelsby

bruecksen commented 1 month ago

Ich würde alle Bugs im Frontend fixen. Wenn ich mich recht erinnere, ist das nur das produktgewicht auf der produktionstagkachel und im Sortiment

Die Grundpreisangabe muss immer in KG geschehen. Daher lass ich dass so. Meintest du das damit?

madmooose commented 1 month ago

Aha, OK. Weil im Supermarkt ist die ja auch manchmal in 100g. Aber dann lass erstmal so lassen. Das können wir ja wann anders nochmal angehen. <- closed

Am 29. Mai 2024 09:30:46 MESZ schrieb "Matthias Brück" @.***>:

Ich würde alle Bugs im Frontend fixen. Wenn ich mich recht erinnere, ist das nur das produktgewicht auf der produktionstagkachel und im Sortiment

Die Grundpreisangabe muss immer in KG geschehen. Daher lass ich dass so. Meintest du das damit?

-- Reply to this email directly or view it on GitHub: https://github.com/bruecksen/bakeup/pull/469#issuecomment-2136725802 You are receiving this because you were mentioned.

Message ID: @.***> -- mobil gesendet Niels Göttsch Gut Hemmelmark 5 24360 Barkelsby

bruecksen commented 1 month ago

Aha, OK. Weil im Supermarkt ist die ja auch manchmal in 100g. Aber dann lass erstmal so lassen. Das können wir ja wann anders nochmal angehen. <- closed

Yo ich glaube shops halten sich ggf. einfach nicht ganz korrekt daran? (hier https://www.haendlerbund.de/de/ratgeber/recht/3766-grundpreisangabe#angabe)