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.

Sign up for GitHub

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

Jump to bottom

Vise om bruker har OS på BT vilkårsvurdering #2764

Merged
merged 14 commits into from
May 30, 2024
Merged

Vise om bruker har OS på BT vilkårsvurdering #2764

merged 14 commits into from
May 30, 2024

Conversation

gunnsteingarmo
Copy link
Contributor

@gunnsteingarmo gunnsteingarmo commented May 15, 2024 •
edited

Hvorfor er denne endringen nødvendig? ✨

Hvis det finnes en periode med OS vises info i en tabell.

  • Lagt til visning av siste periode med OS under aktivitet på BT - endret navn på GrunnbeløpInfo slik at navnet på komponenten blir mer korrekt.
  • Laget funksjonen formaterTilIsoDatoFraTilStreng og tatt denne i bruk der samme kode ble repetert.
  • Sender med grunnlag som props og bruker data periodeHistorikkOvergangsstønad slik at det som vises frem stemmer overens med data på det gjeldende tidspunktet.
  • Sender med behandlingOpprettet som props i stedet for å bruke context og nøste opp i Ressurs
  • Laget hooken useHentAndelHistorikkPerioder sånn at koden enkelt kan gjenbrukes (Brukte den i første versjon).

PR backend
PR backend

Favro

Bildet viser at søker har hatt periode med OS og at det er samordning
image

Bildet viser uten samordning:
image

@gunnsteingarmo
Laget hooken useHentAndelHistorikkPerioder sånn at koden enkelt kan g…
f7188e4 
…jenbrukes. Laget hooken useHentFagsakIder for å enkelt hente fagsaker. Lagt til visning av siste periode med OS under aktivitet på BT - endret navn på GrunnbeløpInfo slik at komponenenten i større grad heter det det gjør. Laget funksjonen formaterTilIsoDatoFraTilStreng og tatt denne i bruk der samme kode ble repetert.
@gunnsteingarmo gunnsteingarmo requested a review from a team as a code owner May 15, 2024 10:36
@gunnsteingarmo
Sender med grunnlagsdata og bruker verdier fra tidligere historikk om…
ee8fd49 
… OS i EF sånn at data som vises frem er det samme som gjaldt på det bestemte tidspunktet.
@gunnsteingarmo gunnsteingarmo mentioned this pull request May 15, 2024
Merged
@gunnsteingarmo gunnsteingarmo mentioned this pull request May 23, 2024
Merged
gunnsteingarmo and others added 8 commits May 28, 2024 10:28
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented May 30, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarCloud

@gunnsteingarmo gunnsteingarmo merged commit 7ddc77e into main May 30, 2024
4 checks passed
@gunnsteingarmo gunnsteingarmo deleted the vise-siste-periode-os-på-bt-aktivitet branch May 30, 2024 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@charliemidtlyng charliemidtlyng charliemidtlyng approved these changes

@ViktorGSolberg ViktorGSolberg Awaiting requested review from ViktorGSolberg

@olekvernberg olekvernberg Awaiting requested review from olekvernberg

@ma10s ma10s Awaiting requested review from ma10s

@throndi throndi Awaiting requested review from throndi

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@gunnsteingarmo @charliemidtlyng