NAV-23269: Legger til tidslinje fra KS sak i familie-felles #845
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
5 tasks
thoalm
added a commit
to navikt/familie-ks-sak
that referenced
this pull request
Nov 22, 2024
### 💰 Hva skal gjøres, og hvorfor? Favro: https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-23269 Trekker tidslinjekode ut i et eget bibliotek slik at man på sikt kan bruke en felles løsning for både BA og KS. PR for kode som ble lagt til i familie-felles: navikt/familie-felles#845 I noen tilfeller møtte vi på et problem med "smart cast", valgte derfor å legge til "!!" noen plasser. Fra StackOverflow: > A smart cast is only valid when multiple accesses of the same property are guaranteed to return the same value. If the property being accessed is defined in a different module from the access location, the module containing the property can be recompiled separately from the module where it’s accessed, breaking the key requirement of the smart cast. ### 🔎️ Er det noe spesielt du ønsker tilbakemelding om? Nei ### ✅ Checklist _Har du husket alle punktene i listen?_ - [ ] Jeg har testet mine endringer i henhold til akseptansekriteriene 🕵️ - [ ] Jeg har config- eller sql-endringer. I så fall, husk manuell deploy til miljø for å verifisere endringene. - [ ] Jeg har skrevet tester. Hvis du ikke har skrevet tester, beskriv hvorfor under 👇 _Jeg har ikke skrevet tester fordi:_ Flyttet eksiterende tester til biblioteket. ### 💬 Ønsker du en muntlig gjennomgang? - [ ] Ja - [x] Nei
thoalm
added a commit
to navikt/familie-ks-sak
that referenced
this pull request
Nov 26, 2024
Favro: https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-23269 Trekker tidslinjekode ut i et eget bibliotek slik at man på sikt kan bruke en felles løsning for både BA og KS. PR for kode som ble lagt til i familie-felles: navikt/familie-felles#845 I noen tilfeller møtte vi på et problem med "smart cast", valgte derfor å legge til "!!" noen plasser. Fra StackOverflow: > A smart cast is only valid when multiple accesses of the same property are guaranteed to return the same value. If the property being accessed is defined in a different module from the access location, the module containing the property can be recompiled separately from the module where it’s accessed, breaking the key requirement of the smart cast. Nei _Har du husket alle punktene i listen?_ - [ ] Jeg har testet mine endringer i henhold til akseptansekriteriene 🕵️ - [ ] Jeg har config- eller sql-endringer. I så fall, husk manuell deploy til miljø for å verifisere endringene. - [ ] Jeg har skrevet tester. Hvis du ikke har skrevet tester, beskriv hvorfor under 👇 _Jeg har ikke skrevet tester fordi:_ Flyttet eksiterende tester til biblioteket. - [ ] Ja - [x] Nei
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
KS og BA skal på sikt bruke samme bibliotek for tidslinje. Per nå har de begge en intern implementasjon. Dette er første steget for en felles løsning. Legger dermed tidslinje-logikken i familie-felles.