scaarup / aula

Fetches information from Aula about your children
GNU General Public License v3.0
51 stars 13 forks source link

Meebook weekplan service #119

Open AThomsen opened 10 months ago

AThomsen commented 10 months ago

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...

scaarup commented 10 months ago

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.

AThomsen commented 10 months ago

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

AThomsen commented 10 months ago

Sådan dér :-)

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

scaarup commented 10 months ago

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 commented 9 months ago

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.