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

Skrive infotekst om at kun de 100 mest aktive får utdelt lodd #49

Open xivind opened 3 months ago

xivind commented 3 months ago

Siden dette er ganske permanent info kan det kanskje være nyttig å skrive det rett i markdown filene og ikke i variabelen for brukerdefinert tekst @thomiz ? Kan kanskje være nyttig å ha denne setningen i begge markdown filer, og ikke bare infofilen?

thomiz commented 3 months ago

Enig, dette er lagt til. Please review @xivind . Infoen er bare lagt til i docs/info.md

xivind commented 3 months ago

Teksten i docs/info.md ser bra ut @thomiz Men det kan hende at ikke alle får med seg den siden den ligger som et kulepunkt sammen øvrig info. Det er jo ganske viktig info, så jeg foreslår at vi i tillegg legger den på resultatsiden, se forslag til plassering i bildet under. Teksten kunne vært slik: 'Denne våren prøver vi ut om vi skal ha vår egen aktivitetskampanje i Helsedirektoratet. Det er foreløpig noen begrensninger i løsningen som gjør at dersom mer enn 100 personer oppnår 150 minutter aktivitet en uke, er det bare de 100 mest aktive som får tildelt lodd for den uken.' Dersom du er enig, gi beskjed, så kan jeg fikse det, og dersom du tenker det er godt nok som det er, bare lukk saken.

Image

thomiz commented 3 months ago

Jeg er usikker, tror det avhenger veldig av hvor mange som faktisk blir aktive deltakere i aksjonen. Hvis det blir mange deltakere bør vi legge denne informasjonen inn på forsiden.

xivind commented 3 months ago

Det er en god vurdering @thomiz . Jeg er enig. Da stenger jeg denne nå, så løfter vi det opp på fremsiden senere, om vi ser at vi blir over 100 aktive.

thomiz commented 3 months ago

Jeg tror vi kan løse dette med å splitte opp i to klubber og skrape begge, vi har allerede testet denne funksjonen @xivind TestBuild. Det betyr at teksten kanskje skal endres til at medlemsmassen må fordele seg på to klubber for å sikre at alle resultater kommer med?

xivind commented 3 months ago

Ja, det stemmer @thomiz Jeg husker det nå! Når du testet med TCK. I så fall er vel kun nødvendig med endringer i scraperen? Teksten må helt klart endres for å få dette frem. Kan vi ikke diskutere dette kort etterpå i dag, hvordan vi kommuniserer dette? Vi må være klare, men vi trenger vel kanskje først å kommunisere denne muligheten når vi f. eks passerer 80?