Skip to content

Commit

Permalink
🚀 Skriv over på ny brukernotifikasjonsløsning (#832)
Browse files Browse the repository at this point in the history
* WIP: Skriv over på ny brukernotifikasjonsløsning

* WIP Lage DTOer for varselhendelser

* Mer Brukernotifikasjons oppdatering

* Gjenbruk

* Mer fiksing av tester

* 🚧 WIP tester

* Fix tester i Brukernotifikasjon

* ??

* 🚧 WIP tester

* BrukerVarselHendelse test kotlin

* Fine-grained modelering av eventer

* Legg til feilmelding på mottatt brukernotifikasjon

* Fix litt på testene og filtrer vekk varsler fra andre apper

* Fjern tidspsunkt fra varsel-dto, fiks delingAvCvTest

* Revert mål-historikk endringer

* Vent med mål-historikk

* Prøv å fikse varsel-hendelse tester

* Endre på spørring og consumer + div

* Fjern not-null assert, returner hvor mange notifikasjoner som ble sendt

* Fix tester i rekrutteringsbistandconsumer

* Set kafka auto-commit interval til 100ms for kafkalisteners

* Rydde litt i tester og logikk

* Fjern ubrukt fil

* Fjern ubrukte brukernotifikasjon schemas

* Bedre typing av varselId, fjern ubrukt kode i ekstern-status SENDT

* Fix noen code smells

* Legg til varsel-hendelse topic

* Legg til namespace på varsel-topics

* Bedre logging

* Droppe et par ubrukte kolonner i brukernotifikasjon

* Rydde bort dobbel dao

* Fjern schema fra migration

* Rename MinSideBrukernotifikasjonsId til MinSideVarselId

* Logg interne-varsel hendelser også

* Fiks varseltype enum

* Bedre loggmelding

* Bedre loggmeldinger på varsel

* Rename service and producer brukertifikasjon -> minsideVarsler

* Mer riktig logging

* Fjern duplikat logging

* Bedre loggmeldinger

* Rename varsel-cron, logg at varsel lagres i outboc

* Bedre tracking av mote sms

* Logg aktivitet-er og (intern) varsel type

* Make param type uuid instead of string

* Add kanal to sendt hendelse in tests

* Log varsel sendt på producer som info ikke debug

* Fiks metrikker, legg til for alle hendelser

* Fiks counter i test

* Reset counter between tests

* Fiks logging av varselId

* Ikke registrer med feilmelding på metrikker uten feilmelding

* Legg til varsel-type på metrikker

* Lag metrikk for tid-til-inaktivering

* Bruk const for ekstern-status-eventName

---------

Co-authored-by: johannetronstad <[email protected]>
Co-authored-by: Hans Petter Simonsen <[email protected]>
  • Loading branch information
3 people authored Oct 23, 2024
1 parent b1f7b89 commit d1fa74c
Show file tree
Hide file tree
Showing 69 changed files with 1,790 additions and 1,945 deletions.
3 changes: 1 addition & 2 deletions build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,6 @@ repositories {

tasks.generateAvroJava {
source(
"src/main/java/no/nav/veilarbaktivitet/brukernotifikasjon/kvittering",
"src/main/java/no/nav/veilarbaktivitet/stilling_fra_nav"
)
}
Expand Down Expand Up @@ -162,7 +161,7 @@ dependencies {
implementation("no.nav.common:util:$common_version")
implementation("no.nav.common:types:$common_version")
implementation("net.sourceforge.collections:collections-generic:4.01")
implementation("com.github.navikt:brukernotifikasjon-schemas:v2.5.2")
implementation("no.nav.tms.varsel:kotlin-builder:1.0.0")
implementation("no.nav.poao.dab:spring-auth:$dab_common_version")
implementation("no.nav.poao.dab:spring-a2-annotations:$dab_common_version")

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,9 @@
import no.nav.veilarbaktivitet.aktivitet.domain.AktivitetTransaksjonsType;
import no.nav.veilarbaktivitet.aktivitet.dto.AktivitetDTO;
import no.nav.veilarbaktivitet.aktivitet.mappers.AktivitetDTOMapper;
import no.nav.veilarbaktivitet.brukernotifikasjon.BrukernotifikasjonService;
import no.nav.veilarbaktivitet.brukernotifikasjon.MinsideVarselService;
import no.nav.veilarbaktivitet.brukernotifikasjon.VarselType;
import no.nav.veilarbaktivitet.brukernotifikasjon.opprettVarsel.AktivitetVarsel;
import no.nav.veilarbaktivitet.person.Innsender;
import no.nav.veilarbaktivitet.person.Person;
import org.springframework.http.HttpStatus;
Expand All @@ -30,7 +31,7 @@ public class AvtaltMedNavService {
private final AktivitetDAO aktivitetDAO;
private final ForhaandsorienteringDAO fhoDAO;
private final MeterRegistry meterRegistry;
private final BrukernotifikasjonService brukernotifikasjonService;
private final MinsideVarselService brukernotifikasjonService;

public static final String AVTALT_MED_NAV_COUNTER = "aktivitet.avtalt.med.nav";
public static final String AKTIVITET_TYPE_LABEL = "AktivitetType";
Expand All @@ -43,7 +44,7 @@ public AvtaltMedNavService(MetricService metricService,
AktivitetDAO aktivitetDAO,
ForhaandsorienteringDAO fhoDAO,
MeterRegistry meterRegistry,
BrukernotifikasjonService brukernotifikasjonService) {
MinsideVarselService brukernotifikasjonService) {

this.metricService = metricService;
this.aktivitetDAO = aktivitetDAO;
Expand Down Expand Up @@ -75,7 +76,8 @@ public AktivitetDTO opprettFHO(AvtaltMedNavDTO avtaltDTO, long aktivitetId, Pers
if(!brukernotifikasjonService.kanVarsles(aktorId)){
throw new IllegalStateException("bruker kan ikke varsles");
}
brukernotifikasjonService.opprettVarselPaaAktivitet(aktivitetId, avtaltDTO.getAktivitetVersjon(), aktorId, FORHAANDSORIENTERING_DITT_NAV_TEKST, VarselType.FORHAANDSORENTERING);
brukernotifikasjonService.opprettVarselPaaAktivitet(
new AktivitetVarsel(aktivitetId, avtaltDTO.getAktivitetVersjon(), aktorId, FORHAANDSORIENTERING_DITT_NAV_TEKST, VarselType.FORHAANDSORENTERING, null, null, null));
}

var nyAktivitet = aktivitetDAO.hentAktivitet(aktivitetId)
Expand Down

This file was deleted.

Loading

0 comments on commit d1fa74c

Please sign in to comment.