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

Refactor utbetalinger #1068

Merged
merged 117 commits into from
Feb 27, 2025
Merged

Refactor utbetalinger #1068

merged 117 commits into from
Feb 27, 2025

Conversation

toresbe
Copy link
Contributor

@toresbe toresbe commented Jan 23, 2025

Omfattende refaktor og forenkling av src/utbetalinger

Ingen vits i å bygge opp setninger fra delsetninger
* Henter inn metode fra utbetalingerUtils som bare ble brukt her
* Endrer metoden så vi ikke må legge på årstall for hånd
* Bedre navn på argumentene
* Destrukturering av props og inline av definisjon
.at(-1) kom i ES2022 og er penere.
Flytter ingress tilbake opp et hakkˆ
Samme funksjon var duplisert to steder dypt i kode, så jeg trakk ut til en egen funksjon
Wrapper dem i en describe, gir mer konvensjonelle navn, fjerner en duplisert test
* .replace er ikke nødvendig; DateTimeFormat legger punktum på ordinaltall
* syntaksendringer (disse funksjonene kommer ikke til å leve lenge)
Fremdeles noen bugs her, så dette bør anses som en checkpoint commit
Forebygger logg-spam under test
Forenklementell inkrementering
…ctor/utbetalinger

# Conflicts:
#	src/utbetalinger/filter/FilterProvider.tsx
#	src/utbetalinger/filter/lib/FilterContext.ts
#	src/utbetalinger/filter/lib/filterLogAnalytics.ts
#	src/utbetalinger/filter/lib/filterMatch.test.ts
#	src/utbetalinger/filter/lib/filterMatch.ts
#	src/utbetalinger/filter/lib/filterReducer.test.ts
#	src/utbetalinger/filter/lib/filterReducer.ts
#	src/utbetalinger/filter/lib/filterResponses.ts
@toresbe toresbe changed the base branch from master to main February 13, 2025 11:29
# Conflicts:
#	src/pages/_app.tsx
#	src/utbetalinger/beta/utbetalinger.module.css
@toresbe toresbe merged commit cfd9b47 into main Feb 27, 2025
5 checks passed
@toresbe toresbe deleted the refactor/utbetalinger branch February 27, 2025 11:56
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