Scouterna / skojjt

Närvarorapportering för scouter
Apache License 2.0
14 stars 15 forks source link

Märkestagning i Skojjt #136

Closed tobbee closed 3 years ago

tobbee commented 3 years ago

Alla datatyper och mappningar mot ndb som behövs borde vara på plats, men man kan putsa lite på en del av grafiken.

Centrala mallar för flera kårer kan också läggas till om det är önskvärt. Eftersom endast bevismärken har klara krav bör de dock alltid gå att skriva över av kåren själv, tänker jag mig.

Löser #124

tobbee commented 3 years ago

@martin-green Här är några saker som kan behöva förbättras som du kan tänka på när du tittar på denna PR: 1) Mer feedback när man sparar märkesprogress på avdelningssida. Just nu kommer inte knappen tillbaks till rätt tillstånd. Kanske skall man lämna sidan efter att den är sparad? 2) Liknande problem med uppdatering av märkesprogress för en specifik scout 3) Snyggare uppladdning av json-definition/bild. Texten på knapparna är nu på engelska. 4) Bilder är nu data-url'er vilket gör att de inte kan cachas, utan på varje sida där bilderna är med så kommer all data att laddas ner. Kan vara OK just nu, eftersom bilden bara är på sidan där märket definieras, men om vi vill ha samma bild på fler ställen vore det bättre att den verkligen är en bild med egen URL. Kan vara lättare att fixa efter att i bytt databas. 5) Kan vara lite väl många loggutskrifter, framförallt i Javascript koden. 6) Lite väl många commits, så kunde vara bra att slå ihop dem 7) Bilder hör inte till stoutkår just nu. Borde de göra det så att det är lättare att spåra dem om det finns olika bilder med samma namn? 8) Centrala mallar finns inte, men borde vara ganska lätt att lägga till med samma mekanism som för kåren märken, men då begränsat till admin. Gäller framförallt bevismärken där det finns relative klara krav. Sen borde man kunna välja dessa som startpunkt för en kår 9) Om vi vill byta key för scout_group till kårid, ska vi göra det innan vi drar igång detta? Likaså med att byta från personnummer till medlemsnummer? Annars blir det ytterligare några entries att byta i en sådan övergång. 10) Dokumentation på Wikin. Det behövs dokumentation, och det kan jag förstås jobba på också, men den är ett eget repo.

TobbeEdgeware commented 3 years ago

Tycker att det känns klart för större demo och sen att någon testkör. Alla kan nu göra och ändra i mallar och märken, men man kan aldrig få färre delar än vad man haft tidigare. Man kan inte heller ändra de delar som är godkända.

Ska titta lite på dokumentationen i Wiki nu.

tobbee commented 3 years ago

Nu har jag lagt upp den här på min kår med länkar till bilder istället för att lagra bilderna i URLer.

TobbeEdgeware commented 3 years ago

@martin-green Nu har detta körts en vecka på min kår och fungerat bra. Har gjort några mindre ändringare, men jag tycker nu att detta är klart för mergning. Jag kan inte ladda om sidan automatiskt efter att jag skickat data med xhr, men jag slänger upp en skylt och talar om att man kan behöva ladda om sidan. Har också begränsat vem som kan lägga till/ändra märken till kåradmin/admin.

Jag har skrivit lite wiki-text på https://github.com/stgscout/stg-skojjt/wiki. Jag kan skriva något liknande men mer allmänt på skojjts wiki när detta är mergat.

TobbeEdgeware commented 3 years ago

En liten sak jag ändrat är att man inte kan dubbelklicka och lägga till samma moment 2ggr. Det ställde till det lite.