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

Nytt bekreftelsesbrev til arbeidsgiver tilpasset inntektsmelding gjennom nav.no #538

Merged
merged 4 commits into from
Jan 16, 2025

Conversation

mortenoosvik
Copy link
Contributor

@mortenoosvik mortenoosvik commented Jan 14, 2025

Bakgrunn

I forbindelse med ny inntektsmelding gjennom nav.no som først skal tas i bruk for pleiepenger, ønsker vi å informere arbeidsgiver om at de kan sende inn inntektsmelding der.

Jira oppgave: https://jira.adeo.no/browse/TSFF-1039

Løsning

  • Har tatt utgangspunkt i den eksisterende ArbeidsgiverMeldingPDFGenerator og lagd en tilsvarende ArbeidsgiverMeldingNavNoPDFGenerator som henter riktig brevmal.
  • Lagt til feature toggle som styres fra SøknadService.

Notat

Tilsvarende må gjøres i k9-sak-innsyn-api.

Bilder

Beskrivelse av brevet Det genererte brevet
Screenshot 2025-01-14 at 17 23 20 Screenshot 2025-01-17 at 10 24 13

@mortenoosvik mortenoosvik requested a review from a team as a code owner January 14, 2025 16:31
Copy link
Collaborator

@ramrock93 ramrock93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bra jobbet, det ser bra ut 👍

Usikker på om det er nødvendig med en helt ny PDF.
Det er snakk om endring av informasjon i eksisterende PDF?
Hadde nok vært mindre kode å skrive ved å toggle de tekstlige endringene rett i eksisterende PDF.

Hvis tanken er at ArbeidsgiverMeldingNavNoPDFGenerator skal erstatte ArbeidsgiverMeldingPDFGenerator er det fint at den markeres som @Deprecated.

@mortenoosvik
Copy link
Contributor Author

Bra jobbet, det ser bra ut 👍

Usikker på om det er nødvendig med en helt ny PDF. Det er snakk om endring av informasjon i eksisterende PDF? Hadde nok vært mindre kode å skrive ved å toggle de tekstlige endringene rett i eksisterende PDF.

Hvis tanken er at ArbeidsgiverMeldingNavNoPDFGenerator skal erstatte ArbeidsgiverMeldingPDFGenerator er det fint at den markeres som @Deprecated.

Jeg kan markere den @Deprecated. Kunne ha gjort endringen direkte i pdf-en, men tenkte det gjorde ryddejobben enklere når den tid kommer 😄

@mortenoosvik mortenoosvik merged commit 87b6051 into master Jan 16, 2025
5 checks passed
@mortenoosvik mortenoosvik deleted the TSFF-1039-arbeidsgiver-brev-ny-info branch January 16, 2025 13:48
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.

2 participants