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

NAV-24196: Tillatter å sette Vikafossen hvis det er eneste tilgang saksbehandler har #5072

Merged

Conversation

thoalm
Copy link
Contributor

@thoalm thoalm commented Feb 11, 2025

💰 Hva skal gjøres, og hvorfor?

Favro: https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-24196

Når saksbehandler kun har tilgang til enheten Vikafossen skal det være lovt å opprette behandlinger på Vikafossen enheten. Saksbehandler med Vikafossen tilgang har ofte (alltid?) kun tilgang til Vikafossen.

Har omdøpt de gamle metodene med en "Gammel" suffix. De nye metodene her omtrent lik, men endringer relatert til det som er nevnt over.

Har lagt de nye endringene bak en toggle.

🔎️ Er det noe spesielt du ønsker tilbakemelding om?

Nei

✅ Checklist

Har du husket alle punktene i listen?

  • Jeg har testet mine endringer i henhold til akseptansekriteriene 🕵️
  • Jeg har config- eller sql-endringer. I så fall, husk manuell deploy til miljø for å verifisere endringene.
  • Jeg har skrevet tester. Hvis du ikke har skrevet tester, beskriv hvorfor under 👇

Jeg har ikke skrevet tester fordi:

💬 Ønsker du en muntlig gjennomgang?

  • Ja
  • Nei

@thoalm thoalm changed the title NAV-24196: Tillatter å sette Vikafossen hvis det er eneste tilgang sa… NAV-24196: Tillatter å sette Vikafossen hvis det er eneste tilgang saksbehandler har Feb 11, 2025
@thoalm thoalm force-pushed the NAV_24196_tillater_aa_sette_vikafossen_hvis_de_er_eneste_mulighet branch 2 times, most recently from d688532 to 4f07f6f Compare February 11, 2025 15:09
@thoalm thoalm marked this pull request as ready for review February 11, 2025 15:11
@thoalm thoalm requested a review from a team as a code owner February 11, 2025 15:11
@thoalm thoalm force-pushed the NAV_24196_tillater_aa_sette_vikafossen_hvis_de_er_eneste_mulighet branch from 4f07f6f to 1f1e7a4 Compare February 11, 2025 15:12
@thoalm thoalm force-pushed the NAV_24196_tillater_aa_sette_vikafossen_hvis_de_er_eneste_mulighet branch from 1f1e7a4 to 4c72623 Compare February 11, 2025 15:18
Copy link
Contributor

@bragejahren bragejahren left a comment

Choose a reason for hiding this comment

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

Ser veldig bra ut! 🙌

@thoalm thoalm added this pull request to the merge queue Feb 12, 2025
Merged via the queue into main with commit 15b90b0 Feb 12, 2025
8 checks passed
@thoalm thoalm deleted the NAV_24196_tillater_aa_sette_vikafossen_hvis_de_er_eneste_mulighet branch February 12, 2025 08:16
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