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

IS-2621: Add stansdato to stans vurdering #35

Merged
merged 1 commit into from
Oct 4, 2024

Conversation

vetlesolgaard
Copy link
Contributor

@vetlesolgaard vetlesolgaard commented Oct 3, 2024

Legger til muligheten for å sende inn stansdato når man gjør en vurdering av typen stans.

Spørsmål: Det er vel ingen som enda er kommet til en stans vurdering rundt manglende medvirkning? Så her kan vi gå rett på at stansdato skal være obligatorisk for en stans vurdering? 😊

Se tilhørende PR i syfomodiaperson

  • PR i syfomodiaperson må merges før denne

@vetlesolgaard vetlesolgaard force-pushed the IS-2621-Add-stansdato-to-stans-vurdering branch from 8e23850 to f63423a Compare October 3, 2024 07:16
@vetlesolgaard vetlesolgaard force-pushed the IS-2621-Add-stansdato-to-stans-vurdering branch from f63423a to 92dae9d Compare October 3, 2024 07:23
@vetlesolgaard vetlesolgaard changed the title 🚧WIP🚧 IS-2621: Add stansdato to stans vurdering IS-2621: Add stansdato to stans vurdering Oct 3, 2024
@vetlesolgaard vetlesolgaard marked this pull request as ready for review October 3, 2024 07:42
@vetlesolgaard vetlesolgaard requested a review from a team as a code owner October 3, 2024 07:42
@andersrognstad
Copy link
Contributor

Legger til muligheten for å sende inn stansdato når man gjør en vurdering av typen stans.

Spørsmål: Det er vel ingen som enda er kommet til en stans vurdering rundt manglende medvirkning? Så her kan vi gå rett på at stansdato skal være obligatorisk for en stans vurdering? 😊

Se tilhørende PR i syfomodiaperson

  • PR i syfomodiaperson må merges før denne

Stemmer at ingen kan gjøre stans i prod enda, så det er good 👍
Men vi har kanskje noen stans-vurderinger som er gjort i dev uten stans-dato, det vil vel potensielt feile når de hentes opp fra databasen pga https://github.com/navikt/ismanglendemedvirkning/pull/35/files#diff-0894e46ae10f832891662fd2229903a9aec1f85b9ccf24a381564a11a5dc8ebcR55 🤔
Mener jeg laget et db-script for å fikse dette i isarbeidsuforhet så går an å gjøre tilsvarende her hvis det trengs.

@vetlesolgaard vetlesolgaard merged commit 2161e74 into main Oct 4, 2024
5 checks passed
@vetlesolgaard vetlesolgaard deleted the IS-2621-Add-stansdato-to-stans-vurdering branch October 4, 2024 12:09
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