Refactor: move store logic to store

teodorlu commented 5 months ago

Jeg synes det ble mye store-logikk i mikrobloggeriet.serve.

Denne PR-en flytter store-logikken til store. default-doc-list ble da tynn, så jeg tok implementasjonen inn i default-cohort-section.

Vil gjerne høre hva dere synes!

teodorlu commented 5 months ago

Jeg begynte å lese på koden rundt, og gjorde litt med serve/cohort-rss-section også.

Orienterte koden rundt dokumenter og kohorter, som er "entitetene" våre. Funksjonene i mikrobloggeriet.cohort og mikrobloggeriet.doc er laget for å jobbe med kohorter og dokumenter, så da jobber vi "med hårene" med resten av kodebasen.
cohort-rss-section trengte bare ett argument, så nå får den bare ett argument.
Og den har nytt navn, cohort-rss-section til fordel for docs->rss-map.

Disse burde kanskje vært en egen PR.

teodorlu commented 5 months ago

Ser også ut som vi ikke har tester på generert RSS, det hadde vært flott å få inn. Vi kunne gjort noe sånt:

teodorlu commented 5 months ago

Takk for feedback!

iterate / mikrobloggeriet