Refaktorera start-metoden

stefer commented 6 years ago

Start-metoden i main.js är komplex att underhålla för att den är så stor (närmare 500 rader).

Den borde kunna brytas upp.

Den har åtminstone fyra stora olika funktioner:

/start startsidan där alla kårer man får se visas
/start/<sgroup_url> för kåren där avdelningarna visas
/start/<sgroup_url>/troop_url för avdelningen där mötesredovisningen sker
/start/<sgroup_url>/troop_url/key_url med flera funktioner för en avdelning

Om jag förstår flask rätt, så borde man kunna ha varsin funktion för var och en av dessa. Visst, det blir lite gemensam kod som måste återupprepas i vardera (hämta user/kår/avdelnings-info) men jag tror ändå det blir tydligare.

Möjligen kan man tom dela upp /start/<sgroup_url>/troop_url/key_url per key (settings, delete, newperson, dak, sensus)

martin-green commented 6 years ago

Ja, absolut. main.py menar du, den har växt för mycket.

puggan commented 4 years ago

Kanske ska passa på att göra en egen undermapp för controllers (de filer som hanterar url:er)

Scouterna / skojjt

Refaktorera start-metoden #49