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

skal vise valgfritt begrunnelsesfelt dersom bruker svarer nei på om s… #2510

Merged

Conversation

ViktorGSolberg
Copy link
Contributor

…øker tidligere har mottatt andre stønader som har betydning

Hvorfor er denne endringen nødvendig? ✨

Tillater meg også å hente ut innlogget saksbehandler dersom man kjører lokalt og prøver å hente en oppgave med en spesifikk oppgaveId. Tidligere mistet man redigeringsretten til behandlingen sin på grunn av ansvarlig-saksbehandler sjekken.

Tror ikke det er noe som behøves å patches her, da eldre behandlinger vil tilby bruker et valgfritt tekstfelt ved nei-svar.

favro

…øker tidligere har mottatt andre stønader som har betydning
@ViktorGSolberg ViktorGSolberg requested a review from a team as a code owner January 11, 2024 10:06
Copy link
Contributor

@ma10s ma10s left a comment

Choose a reason for hiding this comment

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

En testfeil:
VilkårsregelTest.sjekker at output fortsatt er det samme på json$familie_ef_sak:19
"begrunnelseType" : "VALGFRI", vs "begrunnelseType" : "UTEN",

Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Contributor

@charliemidtlyng charliemidtlyng left a comment

Choose a reason for hiding this comment

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

Ser rett ut for min del, dette!

@ma10s
Copy link
Contributor

ma10s commented Jan 11, 2024

Deployer til preprod nå - tar en kjepp test før merge?

(testet - fungerer!)

@ViktorGSolberg ViktorGSolberg merged commit 66a4e31 into main Jan 11, 2024
5 checks passed
@ViktorGSolberg ViktorGSolberg deleted the tidligere-vedtaksperioder-valgfritt-begrunnelsesfelt branch January 11, 2024 12:26
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