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

fix. bruk bedriftNr som identifikator for arbeidsgivere #1304

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

eirikv
Copy link
Contributor

@eirikv eirikv commented Mar 13, 2025

Arbeidsgivere utfører ikke handlering på vegne av seg selv men på vegne av bedriften. Derfor bør identifikatoren på avtaleparten være bedriftnr istedenfor fnr på innlogget bruker.

Denne endringen henter bedriftNr fra header som blir sendt fra frontend via intercepter i Axios på valgt bedrift.

Dette gjør også at vi kan bruke utførtAv som tenkt for arbeidsgivere på lik linje som for deltaker og veileder.

Arbeidsgivere utfører ikke handlering på vegne av seg selv
men på vegne av bedriften. Derfor bør identifikatoren på
avtaleparten være bedriftnr istedenfor fnr på innlogget bruker.

Denne endringen henter bedriftNr fra header som blir sendt
fra frontend via intercepter i Axios på valgt bedrift.

Dette gjør også at vi kan bruke utførtAv som tenkt for
arbeidsgivere på lik linje som for deltaker og veileder.
@eirikv eirikv requested review from Oddsor, sindredl and Copilot March 13, 2025 12:10
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the identifier used for employers by retrieving bedriftNr from the URL and adding it to Axios request headers instead of relying on the user's fnr.

  • Updated Axios interceptor logic in two service files to extract bedriftNr from the URL.
  • Allows consistent use of utførtAv for employers similar to the approach for participants and veileder.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/services/use-rest.ts Added an Axios interceptor to extract bedriftNr from URL parameters and add it to request headers.
src/services/rest-service.ts Added similar Axios interceptor logic alongside axiosRetry configuration.

@eirikv
Copy link
Contributor Author

eirikv commented Mar 13, 2025

Dette funker ikke helt som forventet. Vi må kanskje mellomlagre bedriftnr i session-storage. 🤔

@eirikv eirikv marked this pull request as draft March 13, 2025 14: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.

None yet

1 participant