aula icon indicating copy to clipboard operation
aula copied to clipboard

Meebook weekplan service

Open AThomsen opened this issue 2 years ago • 5 comments

Dette tilføjer en service, lidt alá calendar.list_events, der returnerer meebook weekplan-data for et givent tidspunkt.

Planen er hver aften at bruge TTS til at fortælle hvilke lektier etc. der skal være lavet til dagen efter.

image

Tanker...

  • Jeg kan forstå, at "ugeplaner" er kan betyde flere ting, og at forskellige skoler anvender det på forskellige måder. Forsøget her er baseret på mine egne børns data.

  • Servicen tager sensor som parameter. Entity-pickeren vil så vise alle sensorer, hvilket ikke er optimalt. Idéer til anden tilgang modtages gerne!

AThomsen avatar Oct 17 '23 15:10 AThomsen

Tak for dit bidrag.

Lige et par ting: Det her med at parse "fritekst" datetime felter - "mandag 2. okt."... Kunne man ikke gøre det på en anden måde? Data bliver hentet således: get_payload = '/relatedweekplan/all?currentWeekNumber='+week+'&userProfile=guardian&childFilter[]='+childFilter+'&institutionFilter[]='+institutionFilter Du kender 'week', så du burde jo faktisk vide, at hvis du spørger på "denne uge", så ville første element i weekPlan array'et have date fra i mandags, næste tirsdag osv.? Hvis det er korrekt, så behøver du ikke parse tekst til date.

Når man kalder din service, kan man angive tider hvortil vi ikke har noget data. Vi henter jo kun indeværende og næste uge. Vil du have noget fejlhåndtering på det?

Sidste ting. Alt hvad der udelukkende relaterer sig til Meebook i client.py, skal gerne høre hjemme under if "0004" in self.widgets: og for sensor.py må der implementeres en lignende logik. Jeg har fx ikke meebook men jeg får alligevel den service - det burde jeg ikke.

scaarup avatar Oct 18 '23 06:10 scaarup

Tak for input. Jeg ser på det hurtigst muligt 👍

AThomsen avatar Oct 21 '23 15:10 AThomsen

Sådan dér :-)

Hvad går dette i øvrigt ud på; hvorfor ikke bare anvende fulde navn? person["navn"].split()[0]

AThomsen avatar Oct 23 '23 20:10 AThomsen

Sådan dér :-)

Hvad går dette i øvrigt ud på; hvorfor ikke bare anvende fulde navn? person["navn"].split()[0]

Er det ikke bare navnet på en key i en dict? Kan ikke lige huske det :)

scaarup avatar Oct 24 '23 06:10 scaarup

Er det nødvendigt at omdøbe eksisterende logik, funktioner og variabler? Kan vi holde det her eksklusivt til Meebook brugerne? Fx virker ugeplaner for næste uge ikke længere. Jeg har ugeplaner fra Min Uddannelse.

scaarup avatar Oct 31 '23 08:10 scaarup