search

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

Første dag i kampanjen bør stjernemerking ikke benyttes? #42

Closed thomiz closed 3 months ago

thomiz commented 3 months ago

Alle utøvere får stjernemarkering den første dagen, det gir kanskje ikke så mye verdi?

xivind commented 3 months ago

Jeg personlig likte stjernene, som et slags velkommen til kampanjen, men ser også ditt poeng! Rankingfunksjonen er modifisert slik at det ikke vises symboler i første uke. Det er denne setningen som sørger for det if int(CAMPAIGN_WEEK_START) == int(toolbox.get_current_week_number()): rankings.update({value["athlete_name"]: ""})

Alt ligger i funksjonen get_changed_ranking(self) Tar du en kjapp titt @thomiz og lukker dersom du synes logikken stemmer? Har testet og ser riktig ut, men godt med to par øyne, alltid!

thomiz commented 3 months ago

Med nærmere ettertanke er det kanskje bare den aller første dagen vi ikke skal ha stjerner? Vet du hva, jeg tror vi egentlig bør fjerne denne logikken. Det er jo dumt om vi ikke skal ha stjernemarkering den første uken. Jeg tror du har rett: "personlig likte stjernene, som et slags velkommen til kampanjen".

Du bestemmer, om vi skal gjøre det om eller ikke @xivind

xivind commented 3 months ago

Takk @thomiz Da velger jeg at stjernene kommer tilbake, siden jeg liker litt slike ting, og dessuten stemmer da brukerdokumentasjonen vår, nemlig: at nye utøvere er stjernemarkert ⭐. Det ekstra filteret er fjernet, og logikken er nå slik den var før. Tidligere reviewet, derfor stenger jeg denne nå.