Skip to content

Event keyinfo

Sigurd Tullander requested to merge event_tldr into main

Closes #63 (closed).

Bygger på og skal merges inn i !270 (merged). Men jeg bare gjør dette som en egen mini-MR først, siden det er så mange momenter jeg er usikker på her, så jeg tror det er hensiktsmessig og ta dette som sin egen del først. Det er ikke så hensiktsmessig å merge rett til main, fordi den er ganske sammenvevd med !270 (merged).

Spesifikke ting jeg er usikker på:

  • Er TL;DR-seksjon egentlig et bra navn, eller skal vi kanskje kalle det "Nykelinfo" isteden?
  • Jeg tror det kan være ryddig å vise hele TL;DR på EventList og ikke noe av innhald, istedenfor det litt snåle truncatechars-innhaldet som ikke tar hensyn til linker og markdown-syntax. Er du enig? Fordi hendingar som ikke har noen TL;DR-seksjon kommer jo til å se uhensiktsmessig tomme ut...
  • Kom gjerne med annen pirk og!

Om den nye ordering-test-metoden

Ville bare ha en enkel ordering-test for den nye modellen, i samme stil som EmbeddableText, men det ble litt komplisert med 2 forskjellige fields å sortere på. Dessuten var det egentlig ingen garanti for at ikke rekkefølgen som kom ut bare var et resultat av rekkefølgen de ble lagd i. Denne nye generiske metoden løser dette problemet med shuffling og enumeration og gjør det flere ganger, og gir alt i alt veldig god sikkerhet for at rekkefølgen er definert av de fieldsa og ikke noe annet. Den gjør det faktisk imponerende enkelt og intuitivt å lage robuste tester for ordering, om jeg skal si det selv. Nesten som svart magi.

TODO:

  • Skrive tester
  • Re-target til main
Edited by Sigurd Tullander

Merge request reports