iterate / mikrobloggeriet

1 stars 1 forks source link

Cohorts navigation bar v1 #20

Closed olavm closed 1 year ago

olavm commented 1 year ago

Implementert første utkast av en general navigasjonsbar for kohortene.

Dette er endret:

Ting å vurdere:

For øyeblikket bruker kun mikrobloggene OJ og GENAI denne funksjonaliteten.

Videre må vi implementere og avgjøre:

teodorlu commented 1 year ago

I serve.clj generer doc nå en navbar på toppen, med feeling-lucky (🎲), lenke til "/COHORT/", dvs index'en for den aktuelle kohorten (index i seg selv er ikke enda implementert for cohorts) og visning av hvilket blogginnlegg man leser for øyeblikken.

👍

I cohorts.clj har jeg lagt på sortering, slik at blogginnlegg for kohortene blir presentert i riktig rekkefølge.

(sort-by :doc/slug docs) vil vel ikke funke når vi får flere siffer i tallene våre? Typ sammenlikne 1, 2, 10 og 20.

La til et forslag på "hent tall fra dokument" i #21.

Vi kan gjerne merge denne as-is, og fikse sortering etter at både denne PR-en og #21 er merget.

Laget 2 test blogginnlegg til for OJ, for å teste rekkefølge på visning, og på sikt prev og next funksjonalitet i navbaren.

👍

Da det ikke finnes en variable for cohort name har jeg gjort noen "hacky" løsninger, bla. hente navnet fra cohort :cohort/id i doc, og sortere på :doc/slug i docs.

Forslag: vi lager mikrobloggeriet.cohort/name (funksjon fra kohort til navn). Default-implementasjon leser :cohort/id. Så kan vi endre implementasjonen på kun funksjonen hvis kohorter skal ha navn ulikt id.

Tanker?

Som over: vi kan gjerne merge denne sånn som den er, og fikse småting i ny PR.

teodorlu commented 1 year ago

Godkjenner -- synes det er helt OK å merge nå.

Navn er kanskje noe vi må fikse. Nummer vi fikse, men kan ta det i neste PR.