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-5952 fjerne ubrukt legacy-abac #1415

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

jolarsen
Copy link
Collaborator

Fjerner legacy-kode som nå er ubrukt. Inklusive Systembruker-klassen (injects i MQ-konsumenter og produsenter)
Har allerede flyttet koden som slettes inn i fptilbake, avdeling K9

@jolarsen jolarsen requested a review from a team as a code owner February 10, 2025 19:40
Copy link
Collaborator

@espenwaaga espenwaaga left a comment

Choose a reason for hiding this comment

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

Fint!

Synes fremdeles det er litt rom for å forenkle ForeldrepengerAttributter (inneholder navn og verdier) og ForeldrepengerDataKeys med RessursDataKey.

ForeldrepengerDataKeys inneholder kobling mellom data som blir lagt ved og attributtypen i ForeldrepengerAttributter men ser ikke ut til å være i bruk. Enten ta den i bruk eller fjerne implementasjonen av RessursDataKey med enum?

@jolarsen
Copy link
Collaborator Author

Forenkling av attributter blir en egen aktivitet etter en del avklaringer.
Antagelig en expand med ny versjon av AppRessursData - slik at den gamle kan isoleres til k9tilbake

@jolarsen jolarsen merged commit 6f25cdd into master Feb 12, 2025
4 checks passed
@jolarsen jolarsen deleted the feature/TFP-5952-contract-slett-ubrukt-abac branch February 12, 2025 07:27
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