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

feat(TFP-5687): fjerner ubrukte endepunkt. #2631

Merged
merged 2 commits into from
Feb 26, 2025

Conversation

mrsladek
Copy link
Collaborator

@pekern @jolarsen jeg fant seks endepunkter i abakus som ikke brukes noe sted i fp-sak - mulig noe som k9 bruker som vi bare kan rydde opp i. Flott om dere kan kjøre en raskt QA over dette. Om 👍🏻 så fjerner jeg i neste runde.

@mrsladek mrsladek requested a review from a team as a code owner February 25, 2025 18:10
@mrsladek mrsladek requested review from pekern and jolarsen February 25, 2025 18:10
@mrsladek
Copy link
Collaborator Author

Det finnes en del endepunkter i ForvaltningRestTjeneste som jeg ikke er sikker om vi fortsatt trenger.

  • /settVarigEndring
  • /eliminerInntektsmelding
  • /oppdaterAktoerId

Copy link
Collaborator

@jolarsen jolarsen left a comment

Choose a reason for hiding this comment

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

Petter og du har mer koll på inntektmeldinger og evt behov.

@jolarsen
Copy link
Collaborator

Hør med Petter og Cecilie om hvilke forvaltningstjenester som trengs
Denne /oppdaterAktoerId trengs ved aktør-splitt / merge. Skjær svært sjelden - men når det treffer oss så trengs den.

Copy link
Contributor

@pekern pekern left a comment

Choose a reason for hiding this comment

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

Jeg aner ikke hva de ulike referanse endepunktene har vært brukt til, må være noe historisk jeg ikke vet om. Slett i vei 👍

@mrsladek mrsladek changed the title feat(TFP-5687): markerer ubrukte endepunkt som deprecated. feat(TFP-5687): fjerner ubrukte endepunkt. Feb 26, 2025
@mrsladek
Copy link
Collaborator Author

@pekern @jolarsen jeg fjerner bare det ubrukte med en gang. Ting virker fortsatt ok lokalt og i autotest.

Neste steg er å forenkle kontrakten siden f.eks GrunnlagVersjon ikke brukes for noe annet enn SISTE.

@jolarsen
Copy link
Collaborator

@mrsladek du kan ta de enkle forenklingene i kontraktene, men den store forenklingen (data kun for bruker) må vente på nytt vedtakslager. Så du får vurdere hvor mye tuning det er verdt å gjøre nå.

@mrsladek mrsladek merged commit 736f128 into master Feb 26, 2025
11 of 12 checks passed
@mrsladek mrsladek deleted the feat/TFP-5687_fjerning_av_inaktive_grunnlag branch February 26, 2025 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants