hdir / strava-club

Web-scraping tool and presentation of club results on github pages. Summarized results for a period of time and counting tickets for competition prizes.
https://hdir.github.io/strava-club/vaar24
2 stars 0 forks source link

Utvid datastore med to nye nøkkelpar for å gjøre klar for lag #38

Closed xivind closed 3 months ago

xivind commented 3 months ago

Legge til følgende to nøkler i datastore: "team":"", "athlete_id";"athlete_id"

@thomiz Jeg foreslår å legge til disse to nøklene over i datastore i databehandler. Det tar fem minutter og har ingen ringvirkninger, men det vil gjøre ting veldig mye enklere for oss om vi ser at vi får til det med lagdeling i MVP, enten via Strava Clubs eller FastAPI. Dersom vi ikke får det til blir nøkene bare liggende ubrukt og gjør ingen skade og skaper heller ikke ekstra kompleksitet. Men prinsipielt synes jeg vi skal være enstemmig på endringer i hoveddata-modellen vår. Kan du gi en tommel opp om dette er ok for deg? Eller noe annet om du har innvendinger. PS Jeg vet fokuset vårt er på MVP, men dette tar som sagt 5 min, og det er fakta, og det vil betale seg.

xivind commented 3 months ago

Diskutert i teamet, enstemmig vedtatt, sett i gang 👍👍

xivind commented 3 months ago

De to ekstra nøkkelpar er lagt inn @thomiz , tok en liten prøvekjøring lokalt og som forventet (på akkurat denne) var det ikke noe havari. Vi bruker ikke disse feltene til noe p.t, men dersom vi en vakker høstdag skulle få team og vi fortsatt skrur på denne prototypen er hvert fall alt klart. Slik ser strukturen ut nå. For sikkerhetsskyld setter jeg denne til review, men tror den er ok.

Image

thomiz commented 3 months ago

Dette virker jo bare.