chore(board): transform old line ids to new

[emilielr]

Laget en transformer som mapper alle gamle linjeIDer til nye linjeIDer. Grunnen til at vi tar med både gammel og ny linjeId er slik at vi klarer å hente ut avganger nå for de gamle linjeIDene, og når skiftet skjer (15. desember), så klarer vi å hente ut nye avganger med de nye IDene uten å måtte gjøre noen kodeendringer og "passe på" at det funker. Det som skjer nå er at vi sender både gammel og ny ID inn til journey-planner, og får tilbake avganger på de linjene som faktisk har avganger.

Det var et annet litt mer tricky case som handlet om at de nye linjeIDene er allerede lagt inn som linjer i GraphQL, derav vi får to av samme linjer i prod nå:

Det bydde på to litt ulike tilnærminger.
De to som er like er henholdsvis den gamle og den nye linjeIDen.

1. Jeg kunne valgt å filtrert bort de gamle linjeIDene i selve kortet, men da ville man i firebase lagret de nye linjeIDene når man trykket "lagre". Det igjen ville gjort at vi måtte laget en ny transformer andre veien, slik at ny linje ble mappet til gammel linje for å få ut avgangene frem til 15. desember. Da måtte vi hatt to tranformatorer på en måte; en som mapper fra gammel til ny, og en som mapper fra ny til gammel til 15. desember.

2. Nå har jeg gjort det slik at vi frem til SWITCH_DATE (15. desember) filtrer ut de nye linjeIDene slik at de gamle linjeIDene fortsatt blir lagret til Firebase. Når det blir 15. desember, så filtrerer vi heller ut de gamle, og lagrer de nye. Det eneste man trenger å tenke på da er å fjerne hele compatibility.ts-filen en gang etter 15. desember.

Jeg valgte alternativ 2, men ser ingen umiddelbare fordel med noen av de. Gjerne kom med innspill om det kan gjøres bedre!

Nå:

[SelmaBergstrand]

Tenker alternativ 2 er mindre ny kode og derfor mindre sjanse for slurvefeil, så enig i at den er best. evt kunne man kanskje filtrert bort de doble linjene på samme visningsnavn, men det vil jo måtte gjøres på klienten så er bedre sånn her.