Kaffe-diem / kaffediem

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

Implement orderStore #9

Closed kluvin closed 2 weeks ago

kluvin commented 1 month ago

Abstraksjoner over pocketbase som representerer domenemodellen våres. PR inneholder:

  1. En store for som forskjellige komponenter kan lytte til for å få de siste orders'
  2. En måte å skrive en ny order, sammen med order_drink records.

Todo:

Kommentarer:

[1] Det er ideelt om test for orderStore kan være første klient av orderStore.

Taktikken er merge fast and break things, vi har ikke noe i bruk av sluttbrukere enda, så lenge vi lærer er det det viktigste.

kluvin commented 1 month ago

@IldenH denne slicen vil se mest ut som en backend i henhold til arkitekturen, minus faktisk synkronisering med en store. Spesielt også problemet med typing er relevant.

Vil ta eierskap?

kluvin commented 1 month ago
  1. kan for øvrig helst implementeres som en utvidelse av backend (PB) eller være en ekstern backend-tjeneste. Jeg er bare ikke helt sikker på fremgangsmåten der, og er tryggest med nåværende approach for MVP
LilleAila commented 1 month ago

Formatert kode til å passe med #11

LilleAila commented 3 weeks ago
  • [ ] Fullfør tjeneste til å lytte etter ordre, den skal skrive til en orderStore. [1] (stor)
  • [ ] Fullfør implementasjon for å skrive ny ordre. createOrder. [2] (medium)
  • [ ] Tester for pocketbase, det vi får til. [3]. (medium / stor)
  • [ ] Typer for pocketbase, det vi får til. (medium)
  • [ ] Dockerisering av pocketbase slik at vi kan skrive til en lokal database, og ikke forurense prod. ( medium)

Hvor mye av dette er direkte relatert til denne PR? Er det noen grunn til at denne ikke kan merges akkurat nå, og resten gjøres til issues? Er den orderStore du har skrevet klar til bruk, eller har mangler den noen ting?

Dockerifisering er gjort i #43 , og types blir vel generert med makefilen allerede, og trenger bare tas i bruk i en senere PR.

kluvin commented 3 weeks ago

@LilleAila jeg fjernet de som ikke strengt behøves.