Markera licens aktiverad/betald för lyftare

Krisell / gkk-anmalan

Göteborg Kraftsportklubb (GKK) Anmälningsweb

https://anmalan.gkk-styrkelyft.se

0 stars 1 forks source link

Markera licens aktiverad/betald för lyftare #66

Closed wilari932 closed 1 month ago

wilari932 commented 2 months ago

Krisell commented 2 months ago

Svinbra! Jag har inte lusläst koden än och testat helt, men två spontana tankar:

Det verkar inte gå att kryssa ur, utan när man laddar om kommer de ändå tillbaka ikryssade.
Kan vi använda samma "checkbox"-toggle som för de andra avgifterna (se bild)?

Jag tror också att vi kan skippa 2024 och börja använda detta skarpt för 2025, så att kassören inte behöver manuellt lägga till alla för i år.

Jag har planer på att förändra lite kring licens- och avgiftshantering, men har inte riktigt tänkt igenom det än så vi kan definitivt lägga till detta så länge.

wilari932 commented 2 months ago

Hej! Tack för feedbacken. Ja, vi kan definitivt återanvända den befintliga toggle-modulen. Jag tror dock att jag missade att testa koden tillräckligt noggrant och kommer att undersöka varför den inte uppdateras som förväntat.

Hanteringen av detta behöver en mer långsiktig lösning, Men detta kan fungera som en tillfällig snabbfix. Jag kollar närmare på buggen och återkommer så snart jag har en lösning.

wilari932 commented 2 months ago

@Krisell The bug was fixed and now we use the regular toggle :)

Krisell commented 1 month ago

I don't think I can push the suggested changes as the branch lives in your fork. Now that you have access, could you re-push this as a branch of this repo, or just merge and we can make adjustments in master.