Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gjer at vi tåler nye Hendelse-kategoriar #1819

Merged
merged 10 commits into from
Dec 19, 2024
Merged

Conversation

ingfo
Copy link
Contributor

@ingfo ingfo commented Dec 18, 2024

Describe your changes

Verdi:

  • Dab kan starte å sende meldingar på ny kategori, slik at vi kan implementere filter når vi er klare
  • Gjer koden litt meir robust for endringar

Problemstilling:

Koden er enn så lenge ikkje skriven for å tåle ulike hendelseskategoriar. Det kan hende testar knekk, og at eksisterande filter ikkje oppfører seg som venta.

Konklusjon etter utforsking:

hentEldste bryr seg ikkje om typen hendelse, den ser kun på dato og personident.
Oppdatering av eldste hendelse i opensearch vil bli feil dersom:

  • ein forsøker å lagre ein ny eldste Utgått varsel-hendelse, og
  • det finnes ein Usendt samtalereferat-hendelse som er eldre
    Vi oppdaterer ikkje opensearch med andre hendelsestypar enn Utgått varsel.

Løysing

  • Skriv nye testar som viser problema
  • Skrive om hentEldste til å bli hentEldsteUtgattVarsel
  • Skriv om testar slik at utgått-varsel-testar har utgått-varsel-kategori på hendelse

Trello ticket number and link

https://trello.com/c/2Jf2O0Pk/857-gjer-at-vi-kan-ta-i-mot-udelt-samtalereferat-utan-at-utg%C3%A5tte-varsel-f%C3%A5r-feil

Type of change

  • New feature (non-breaking change which adds functionality)

Checklist before requesting a review

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests.

Prodsette: Sortering på Utgått varsel-dato
Legg til rette for å ta i mot nye kategoriar
@ingfo ingfo force-pushed the juster-tester-utgatt-varsel branch 2 times, most recently from 7ca03db to 7cd6e77 Compare December 18, 2024 12:39
@ingfo ingfo force-pushed the juster-tester-utgatt-varsel branch from 7cd6e77 to 7ff7152 Compare December 18, 2024 12:45
@ingfo ingfo marked this pull request as ready for review December 18, 2024 14:15
@ingfo ingfo requested a review from a team as a code owner December 18, 2024 14:15
@ingfo ingfo changed the title Juster tester utgatt varsel Gjer at vi tåler nye Hendelse-kategoriar Dec 19, 2024
ingfo and others added 3 commits December 19, 2024 10:53
og gjort hentEldste meir generell/gjenbrukbar

Co-authored-by: Sondre Larsen Ovrid <sondre.larsen.ovrid@nav.no>
Co-authored-by: Sondre Larsen Ovrid <sondre.larsen.ovrid@nav.no>
@ingfo ingfo merged commit b097887 into dev Dec 19, 2024
9 checks passed
@ingfo ingfo deleted the juster-tester-utgatt-varsel branch December 19, 2024 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants