Kaffe-diem / kaffediem

https://kaffe-diem.web.app
GNU General Public License v3.0
1 stars 1 forks source link

Kitchen route #61

Closed IldenH closed 5 days ago

IldenH commented 1 week ago

Gjør også at bestillingene har finere ID-er basert på index.

ID-ene teller med "dispatched" ordre akkurat nå. orderStore bør nok finne ut ID-ene i stedet for OrderList.

github-actions[bot] commented 1 week ago

Visit the preview URL for this PR (updated for commit df5e7ff):

https://kaffe-diem--pr61-kitchen-display-w5xg4iq5.web.app

(expires Wed, 27 Nov 2024 21:24:17 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 442774dd5e8e2b443bd55c0d98e4d4879d91742b

kluvin commented 1 week ago

nvm, PR er IKKE ready for review. Det er en bug hvor drinks noen ganger er undefined. Hender spesielt når man flytter en ting fra received til production.

LilleAila commented 6 days ago

nvm, PR er IKKE ready for review. Det er en bug hvor drinks noen ganger er undefined. Hender spesielt når man flytter en ting fra received til production.

@kluvin fikset det. Feilen var at du prøvde å passe en expand til Order, uten at denne attributten var implementert...

LilleAila commented 6 days ago

Eneste problem jeg merker er at (i hvertfall på firefox), vil ikke listen oppdateres når jeg flytter noe fra en state til en annen, og krever en reload.

EDIT: Jeg testet på chromium også, og fikk samme resultat. Det bør fikses på et tidspunkt før release