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

TFP-6037: Fjerner alle Oppgitt Opptjening aggregat uten referanse til grunnlag #2642

Merged

Conversation

mrsladek
Copy link
Collaborator

No description provided.

@mrsladek mrsladek requested a review from a team as a code owner March 10, 2025 20:50
@github-actions github-actions bot added the web label Mar 10, 2025
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.

Ser bra ut. Men her er vi inne på et område der jeg er litt usikker på hva som er blitt gjort av patching opp gjennom tiden. Stort sett du, jeg, Petter som har holdt på.
Burde vi sett gjennom bestillingene etter noen rare patterns for å dobbelsjekke?
Så lenge objektene refererer oppover i treet så bør ting være greit.

@mrsladek
Copy link
Collaborator Author

mrsladek commented Mar 11, 2025

Ser bra ut. Men her er vi inne på et område der jeg er litt usikker på hva som er blitt gjort av patching opp gjennom tiden. Stort sett du, jeg, Petter som har holdt på. Burde vi sett gjennom bestillingene etter noen rare patterns for å dobbelsjekke? Så lenge objektene refererer oppover i treet så bør ting være greit.

Svart lite å slette her - hele 2 aggregater som ikke brukes i en aktiv grunnlag, og det er egentlig riktig siden data kommer fra søknad og gjenbrukes stort sett i alle behandlinger på en sak. Jeg tipper at det er kanskje behadlinger hvor det ble overstyrt en del ganger.
Vi kan godt analysere noe mer før vi fjerner noe.

@mrsladek
Copy link
Collaborator Author

@jolarsen jeg sjekket i basen på nytt og det er kun 2 inaktive som kunne slettes... lurer på om vi trenger en daglig jobb her - eller det holder til å trigge den 1 gang i måneden...

@mrsladek mrsladek merged commit 1682714 into master Mar 11, 2025
11 of 12 checks passed
@mrsladek mrsladek deleted the feat/TFP-6037_fjerne_oppgitt_opptjening_uten_referanse branch March 11, 2025 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants