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
Show file tree
Hide file tree
Changes from 116 commits
Commits
Show all changes
117 commits
Select commit Hold shift + click to select a range
19c7830
Fjerner kode som kun ble brukt av sine egne enhetstester
toresbe Jan 17, 2025
0bd20c0
Bruk date-fns i hentMaanedString
toresbe Jan 17, 2025
0239de4
remove braces around arrow function body
toresbe Jan 17, 2025
96760e2
Legger inn tailwind css
toresbe Jan 17, 2025
d28a8d2
Forenkler i18n-strenger
toresbe Jan 17, 2025
4f5a1be
Fjerner default import til fordel for named
toresbe Jan 17, 2025
6e7c53f
Refaktor ManedGruppe
toresbe Jan 17, 2025
ae62087
Refaktor NyeUtbetalinger.tsx
toresbe Jan 17, 2025
91be01f
Bruk mer moderne syntaks i useEffect
toresbe Jan 17, 2025
bf5b525
Bruk const string union og ikke enum til aktiv tab
toresbe Jan 17, 2025
b04eee4
Legger til classnames
toresbe Jan 17, 2025
f018a67
Renamer TidligerUtbetalinger til TidligereUtbetalinger
toresbe Jan 17, 2025
c7a2a52
Trekker addIdToUtbetalinger.tsx ut til egen fil
toresbe Jan 17, 2025
344f4c6
bruker defineConfig i orval config for typesikkerhet
toresbe Jan 17, 2025
c169e15
Ekskluder generert kode fra git-repo, generer kode som et build-steg
toresbe Jan 17, 2025
4c7fbb4
Ah, vi har et run-orval-flagg; da kan vi bare bruke det.
toresbe Jan 17, 2025
361ea12
Generer suspense-klienter med orval
toresbe Jan 17, 2025
94e9535
Tillat ubrukte argumenter dersom de er prefixed med _
toresbe Jan 17, 2025
bb27693
Legger til react query devtools som build dependency
toresbe Jan 17, 2025
130006f
Generaliser ErrorBoundary litt
toresbe Jan 20, 2025
d1a064f
Refaktorerer tester av utbetalingsdetaljerDefaultAapnet
toresbe Jan 20, 2025
a0a8fa8
Refaktor utbetalingsdetaljerDefaultAapnet om til isNotMoreThanTwoWeek…
toresbe Jan 20, 2025
c5a0ba4
Refaktorerer isNotMoreThanTwoWeeksAgo
toresbe Jan 20, 2025
e48b7f2
Erstatter styled-component med div og Tailwind-CSS
toresbe Jan 20, 2025
67f53cc
isLessThanTwoWeeksAgo er et bedre navn enn isNotMoreThanTwoWeeksAgo
toresbe Jan 20, 2025
a50974c
Lager felles error/loading-komponent for nye og tidligere utbetalinger
toresbe Jan 20, 2025
81755f3
Div safe refaktorering, spes. propagate to destructuring
toresbe Jan 20, 2025
b607b9b
Endrer Accordion.Item fra controlled til uncontrolled for ytelse
toresbe Jan 20, 2025
7a5ca79
Forenkler erStoppet/utbetalingsbeløp
toresbe Jan 20, 2025
96dd25a
Refaktor av formattere
toresbe Jan 20, 2025
dd23e2d
Migrerer fra styled-components til tailwind & div refaktor
toresbe Jan 20, 2025
cf559a4
Forenkler i18n for utbetalingsstatus
toresbe Jan 20, 2025
5e3118c
Forenkler UUID-generering og skroter custom datatype
toresbe Jan 20, 2025
a90a4ab
Splitter UtbetalingAccordionHeader ut til egen fil
toresbe Jan 20, 2025
98eb573
Refaktor av utbetalingsmetode for lesbarhet
toresbe Jan 20, 2025
5fa25e2
Fjerner to nivåer med no-op JSX-fragments
toresbe Jan 20, 2025
9ad79d1
Splitter UtbetalingAccordionContent til egen fil, inliner AccordionItem
toresbe Jan 20, 2025
e56bddb
CSS-fixes for UtbetalingAccordionHeader.tsx
toresbe Jan 21, 2025
4909173
Overhaling av utbetaling-filter
toresbe Jan 21, 2025
26f33f5
Refaktor: fjerner useChips, bruker useFilter direkte, splitter ut Fil…
toresbe Jan 21, 2025
3531e75
Ajourfører innsyn-api.json med nåværende innsyn-api (400-respons)
toresbe Jan 23, 2025
abefedc
Checkpoint commit
toresbe Jan 23, 2025
2e8c080
lager en sterkere typet variant av logAmplitudeEvent (#1066)
toresbe Jan 23, 2025
5d7a81d
Oppdater OpenAPI-schema
toresbe Jan 23, 2025
0461ec1
Sjekker inn snapshot
toresbe Jan 23, 2025
8f054e9
Oppdaterer i18n-streng
toresbe Jan 23, 2025
05c4fb2
Oppdaterer Aksel
toresbe Jan 23, 2025
f67c4a5
Merge branch 'master' into refactor/utbetalinger
toresbe Jan 23, 2025
215d3a8
Unngår null-verdi i RadioGroup
toresbe Jan 23, 2025
0a9423b
Fikser i18n-nøkkel-bug
toresbe Jan 23, 2025
24306be
Fjerner måneder uten utbetalinger
toresbe Jan 23, 2025
3b02278
Bedre variabelnavn
toresbe Jan 23, 2025
e4f412a
Flytter data fetching fra UtbetalingerPanel til …Nye
toresbe Jan 23, 2025
8ee81c1
En API-handler skal ikke returnere noe
toresbe Jan 23, 2025
9286c2b
logAmplitudeEventTyped kan forenkles
toresbe Jan 23, 2025
837ef45
Ikke prøv å logge til Amplitude under test
toresbe Jan 23, 2025
169986a
handler skal ikke returnere en verdi
toresbe Feb 3, 2025
1b9e3bf
Fjerner en NOP-fragment
toresbe Feb 4, 2025
af75ed5
Sjekker inn OpenAPI-endringer
toresbe Feb 4, 2025
c79d1c9
Ikke prøv å logge til Amplitude under test
toresbe Feb 11, 2025
165e7cb
Fjerner kode som kun ble brukt av sine egne enhetstester
toresbe Jan 17, 2025
521c5ec
Bruk date-fns i hentMaanedString
toresbe Jan 17, 2025
3ccd365
remove braces around arrow function body
toresbe Jan 17, 2025
09c7567
Forenkler i18n-strenger
toresbe Jan 17, 2025
74e6b9b
Fjerner default import til fordel for named
toresbe Jan 17, 2025
02ec847
Refaktor ManedGruppe
toresbe Jan 17, 2025
36c0a57
Refaktor NyeUtbetalinger.tsx
toresbe Jan 17, 2025
33384fd
Bruk mer moderne syntaks i useEffect
toresbe Jan 17, 2025
b957eef
Bruk const string union og ikke enum til aktiv tab
toresbe Jan 17, 2025
4e9964d
Renamer TidligerUtbetalinger til TidligereUtbetalinger
toresbe Jan 17, 2025
dcc1075
Trekker addIdToUtbetalinger.tsx ut til egen fil
toresbe Jan 17, 2025
a071d53
bruker defineConfig i orval config for typesikkerhet
toresbe Jan 17, 2025
f858797
Ekskluder generert kode fra git-repo, generer kode som et build-steg
toresbe Jan 17, 2025
0c05b9f
Ah, vi har et run-orval-flagg; da kan vi bare bruke det.
toresbe Jan 17, 2025
71cbd75
Generer suspense-klienter med orval
toresbe Jan 17, 2025
0b17918
Tillat ubrukte argumenter dersom de er prefixed med _
toresbe Jan 17, 2025
2dd9133
Legger til react query devtools som build dependency
toresbe Jan 17, 2025
584b9e0
Generaliser ErrorBoundary litt
toresbe Jan 20, 2025
9d9d076
Refaktorerer tester av utbetalingsdetaljerDefaultAapnet
toresbe Jan 20, 2025
3767369
Refaktor utbetalingsdetaljerDefaultAapnet om til isNotMoreThanTwoWeek…
toresbe Jan 20, 2025
5cc0c3a
Refaktorerer isNotMoreThanTwoWeeksAgo
toresbe Jan 20, 2025
c540c2b
Erstatter styled-component med div og Tailwind-CSS
toresbe Jan 20, 2025
b93cf05
isLessThanTwoWeeksAgo er et bedre navn enn isNotMoreThanTwoWeeksAgo
toresbe Jan 20, 2025
b3a5e78
Lager felles error/loading-komponent for nye og tidligere utbetalinger
toresbe Jan 20, 2025
48e4331
Div safe refaktorering, spes. propagate to destructuring
toresbe Jan 20, 2025
a58cdbd
Endrer Accordion.Item fra controlled til uncontrolled for ytelse
toresbe Jan 20, 2025
eb0ed6d
Forenkler erStoppet/utbetalingsbeløp
toresbe Jan 20, 2025
2d6278e
Refaktor av formattere
toresbe Jan 20, 2025
a94d534
Migrerer fra styled-components til tailwind & div refaktor
toresbe Jan 20, 2025
7029961
Forenkler i18n for utbetalingsstatus
toresbe Jan 20, 2025
a530764
Forenkler UUID-generering og skroter custom datatype
toresbe Jan 20, 2025
f93d56a
Splitter UtbetalingAccordionHeader ut til egen fil
toresbe Jan 20, 2025
1c93f49
Refaktor av utbetalingsmetode for lesbarhet
toresbe Jan 20, 2025
7e95744
Fjerner to nivåer med no-op JSX-fragments
toresbe Jan 20, 2025
cf349ac
Splitter UtbetalingAccordionContent til egen fil, inliner AccordionItem
toresbe Jan 20, 2025
c4a2732
CSS-fixes for UtbetalingAccordionHeader.tsx
toresbe Jan 21, 2025
057d505
Overhaling av utbetaling-filter
toresbe Jan 21, 2025
bf234b1
Refaktor: fjerner useChips, bruker useFilter direkte, splitter ut Fil…
toresbe Jan 21, 2025
3a1b280
Checkpoint commit
toresbe Jan 23, 2025
32c92a3
lager en sterkere typet variant av logAmplitudeEvent (#1066)
toresbe Jan 23, 2025
d2f0cac
Sjekker inn snapshot
toresbe Jan 23, 2025
646bab1
Oppdaterer i18n-streng
toresbe Jan 23, 2025
a945ade
Unngår null-verdi i RadioGroup
toresbe Jan 23, 2025
b41bcd1
Fikser i18n-nøkkel-bug
toresbe Jan 23, 2025
31ef887
Fjerner måneder uten utbetalinger
toresbe Jan 23, 2025
822288c
Bedre variabelnavn
toresbe Jan 23, 2025
b8303f1
Flytter data fetching fra UtbetalingerPanel til …Nye
toresbe Jan 23, 2025
e449df1
En API-handler skal ikke returnere noe
toresbe Jan 23, 2025
2ff54b8
logAmplitudeEventTyped kan forenkles
toresbe Jan 23, 2025
102abe7
Ikke prøv å logge til Amplitude under test
toresbe Jan 23, 2025
a42906b
Rename FilterPredicate til FilterCriteria
toresbe Feb 13, 2025
a64526a
Unbreak my merge
toresbe Feb 13, 2025
c45fec9
Merge remote-tracking branch 'origin/refactor/utbetalinger' into refa…
toresbe Feb 13, 2025
e50e410
Merge branch 'master' into refactor/utbetalinger
toresbe Feb 13, 2025
4bd7c80
Endre typenavn fra FilterPredicate til FilterCriteria
toresbe Feb 18, 2025
29bd739
Merge branch 'main' into refactor/utbetalinger
toresbe Feb 18, 2025
ba3b96f
Merge branch 'main' into refactor/utbetalinger
toresbe Feb 26, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions eslint.config.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ export default [
"no-console": "warn",
"@typescript-eslint/explicit-function-return-type": "off",
"prettier/prettier": "warn",
"@typescript-eslint/no-unused-vars": ["error", { argsIgnorePattern: "^_" }],

"import/order": [
"warn",
Expand Down
10 changes: 8 additions & 2 deletions orval.config.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
export default {
import { defineConfig } from "orval";

export default defineConfig({
"innsyn-api": {
input: "./innsyn-api.json",
output: {
Expand All @@ -24,10 +26,14 @@ export default {
fetch: {
includeHttpStatusReturnType: false,
},
query: {
useSuspenseQuery: true,
version: 5,
},
},
},
hooks: {
afterAllFilesWrite: "prettier --write",
},
},
};
});
44 changes: 38 additions & 6 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@
"@eslint/eslintrc": "^3.2.0",
"@eslint/js": "^9.17.0",
"@faker-js/faker": "^8.4.1",
"@tanstack/react-query-devtools": "^5.64.1",
"@testing-library/jest-dom": "^6.6.3",
"@testing-library/react": "^16.1.0",
"@types/jest": "^29.5.12",
Expand Down
18 changes: 9 additions & 9 deletions public/locales/en/utbetalinger.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@
"filter.format":"(dd.mm.yyyy)",
"filter.mottaker":"Select recipient",
"filter.alle":"All",
"filter.minKonto":"To me",
"filter.annen":"To other recipients",
"filter.mottaker.minKonto":"To me",
"filter.mottaker.annenMottaker":"To other recipients",
"filter.ugylding":"The date should be in the format dd.mm.yyyy",
"filter.tidligstFra":"The earliest search date is 15 months back",
"filter.fraEtterTil":"The From date cannot be after the To date",
Expand All @@ -24,13 +24,13 @@
"modal.avbryt": "Cancel",
"modal.vis": "Show matches",
"feil.fetch" : "Something went wrong when we tried to retrieve your payments. Please try again later.",
"feil.ingen":"We could not find any payments",
"feil.ingen.filter":"for selected filtering",
"feil.ingen.default.tidligere":"for the last 15 months",
"feil.ingen.default.nye":"for current or coming months",
"stoppet": "Stopped",
"planlagt": "Planned",
"utbetalt" : "Paid",
"feil.ingen.filter":"We could not find any payments for selected filtering",
"feil.ingen.default.tidligere":"We could not find any payments for the last 15 months",
"feil.ingen.default.nye":"We could not find any payments for current or coming months",
"utbetalingStatus.STOPPET": "Stopped",
"utbetalingStatus.PLANLAGT_UTBETALING": "Planned",
"utbetalingStatus.UTBETALT": "Paid",
"utbetalingStatus.ANNULLERT": "Annulled",
"periode" : "Period",
"mottaker" : "Recipient",
"tilDeg": "To you",
Expand Down
18 changes: 9 additions & 9 deletions public/locales/nb/utbetalinger.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@
"filter.format":"(dd.mm.åååå)",
"filter.mottaker":"Velg mottaker",
"filter.alle":"Alle",
"filter.minKonto":"Til meg",
"filter.annen":"Til andre mottakere",
"filter.mottaker.minKonto":"Til meg",
"filter.mottaker.annenMottaker":"Til andre mottakere",
"filter.ugylding":"Datoen må være i formatet dd.mm.åååå",
"filter.tidligstFra":"Tidligste søkedato er 15 måneder tilbake i tid",
"filter.fraEtterTil":"Fra-dato kan ikke være etter til-dato",
Expand All @@ -24,13 +24,13 @@
"modal.avbryt": "Avbryt",
"modal.vis": "Vis treff",
"feil.fetch" : "Noe gikk galt da vi skulle hente utbetalingene dine. Vennligst prøv igjen senere.",
"feil.ingen":"Vi fant ingen utbetalinger",
"feil.ingen.filter":"for valgt filtrering",
"feil.ingen.default.tidligere":"for de siste 15 måneder",
"feil.ingen.default.nye":"for nåværende eller kommende måneder",
"stoppet": "Stoppet",
"planlagt": "Planlagt",
"utbetalt" : "Utbetalt",
"feil.ingen.filter":"Vi fant ingen utbetalinger for valgt filtrering",
"feil.ingen.default.tidligere":"Vi fant ingen utbetalinger for de siste 15 måneder",
"feil.ingen.default.nye":"Vi fant ingen utbetalinger for nåværende eller kommende måneder",
"utbetalingStatus.STOPPET": "Stoppet",
"utbetalingStatus.PLANLAGT_UTBETALING": "Planlagt",
"utbetalingStatus.UTBETALT": "Utbetalt",
"utbetalingStatus.ANNULLERT": "Annullert",
"periode" : "Periode",
"mottaker" : "Mottaker",
"tilDeg": "Til deg",
Expand Down
18 changes: 9 additions & 9 deletions public/locales/nn/utbetalinger.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@
"filter.format":"(dd.mm.åååå)",
"filter.mottaker":"Vel mottakar",
"filter.alle":"Alle",
"filter.minKonto":"Til meg",
"filter.annen":"Til andre mottakarar",
"filter.mottaker.minKonto":"Til meg",
"filter.mottaker.annenMottaker":"Til andre mottakarar",
"filter.ugylding":"Datoen må vere i formatet dd.mm.åååå",
"filter.tidligstFra":"Tidlegaste søkjedato er 15 månader bak i tid",
"filter.fraEtterTil":"Frå-datoen kan ikkje vere etter til-datoen",
Expand All @@ -24,13 +24,13 @@
"modal.avbryt": "Avbryt",
"modal.vis": "Vis treff",
"feil.fetch" : "Noko gjekk gale då vi skulle hente utbetalingane dine. Prøv igjen seinare.",
"feil.ingen":"Vi fann ingen utbetalingar",
"feil.ingen.filter":"med filteret som er valt",
"feil.ingen.default.tidligere":"for dei siste 15 månadene",
"feil.ingen.default.nye":"for inneverande eller komande månader",
"stoppet": "Stoppa",
"planlagt": "Planlagt",
"utbetalt" : "Utbetalt",
"feil.ingen.filter":"Vi fann ingen utbetalingar med filteret som er valt",
"feil.ingen.default.tidligere":"Vi fann ingen utbetalingar for dei siste 15 månadene",
"feil.ingen.default.nye":"Vi fann ingen utbetalingar for inneverande eller komande månader",
"utbetalingStatus.STOPPET": "Stoppa",
"utbetalingStatus.PLANLAGT_UTBETALING": "Planlagt",
"utbetalingStatus.UTBETALT": "Utbetalt",
"utbetalingStatus.ANNULLERT": "Annullert",
"periode" : "Periode",
"mottaker" : "Mottakar",
"tilDeg": "Til deg",
Expand Down
17 changes: 6 additions & 11 deletions src/components/errors/ErrorBoundary.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -5,35 +5,30 @@ import ServerError from "../../pages/500";

interface Props {
children?: ReactNode;
fallback?: ReactNode;
}

interface State {
hasError: boolean;
error: Error | null;
}

class ErrorBoundary extends React.Component<Props, State> {
constructor(props: Props) {
super(props);

// Define a state variable to track whether is an error or not
this.state = { hasError: false };
this.state = { error: null };
}

/* eslint-disable-next-line @typescript-eslint/no-unused-vars */
static getDerivedStateFromError(_: Error) {
static getDerivedStateFromError(error: Error) {
// Update state so the next render will show the fallback UI

return { hasError: true };
return { error };
}
public componentDidCatch(error: Error, errorInfo: ErrorInfo) {
logger.error(`Uncaught clientside error: ${error}, errorInfo: ${JSON.stringify(errorInfo)}`);
}

public render() {
if (this.state.hasError) {
return <ServerError />;
}

if (!!this.state.error) return this.props.fallback ?? <ServerError />;
return this.props.children;
}
}
Expand Down
10 changes: 4 additions & 6 deletions src/pages/500.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,10 @@ const ServerError = (): React.JSX.Element => (
<div className="informasjon-side">
<FeilsideWrapper>
<UthevetPanel>
<>
<Heading level="1" size="large" spacing>
Beklager, vi har dessverre tekniske problemer.
</Heading>
<BodyLong spacing>Vennligst prøv igjen senere.</BodyLong>
</>
<Heading level="1" size="large" spacing>
Beklager, vi har dessverre tekniske problemer.
</Heading>
<BodyLong spacing>Vennligst prøv igjen senere.</BodyLong>
</UthevetPanel>
</FeilsideWrapper>
</div>
Expand Down
3 changes: 2 additions & 1 deletion src/pages/api/innsyn-api/[...slug].ts
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,8 @@ const handler = async (req: NextApiRequest, res: NextApiResponse<unknown>) => {
}
}
if (!token) {
return res.status(401);
res.status(401);
return;
}
if (!slug) {
res.status(400).json({ message: "Manglende path" });
Expand Down
16 changes: 8 additions & 8 deletions src/pages/utbetaling.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -9,12 +9,12 @@ import { useHentAlleSaker } from "../generated/saks-oversikt-controller/saks-ove
import { useHarSoknaderMedInnsyn } from "../generated/soknad-med-innsyn-controller/soknad-med-innsyn-controller";
import UtbetalingsoversiktIngenSoknader from "../utbetalinger/UtbetalingsoversiktIngenSoknader";
import UtbetalingsoversiktIngenInnsyn from "../utbetalinger/UtbetalingsoversiktIngenInnsyn";
import { FilterProvider } from "../utbetalinger/beta/filter/FilterContext";
import UtbetalingerFilter from "../utbetalinger/beta/filter/UtbetalingerFilter";
import UtbetalingerPanelBeta from "../utbetalinger/beta/UtbetalingerPanelBeta";
import styles from "../utbetalinger/beta/utbetalinger.module.css";
import UtbetalingerFilter from "../utbetalinger/filter/UtbetalingerFilter";
import UtbetalingerPanel from "../utbetalinger/UtbetalingerPanel";
import styles from "../utbetalinger/utbetalinger.module.css";
import useUpdateBreadcrumbs from "../hooks/useUpdateBreadcrumbs";
import pageHandler from "../pagehandler/pageHandler";
import { FilterProvider } from "../utbetalinger/filter/FilterProvider";

import Error from "./_error";

Expand All @@ -38,7 +38,7 @@ const Utbetalinger: NextPage = () => {
if (isAlleSakerLoading || isHarSoknaderMedInnsynLoading) {
return (
<div className={styles.utbetalinger_side} data-theme="utbetalinger">
<Loader className={styles.utbetalinger_loader} size="3xlarge" title="venter..." />
<Loader className="m-[100px]" size="3xlarge" title="venter..." />
</div>
);
}
Expand Down Expand Up @@ -66,13 +66,13 @@ const Utbetalinger: NextPage = () => {
</Head>
<FilterProvider>
<div className={styles.utbetalinger_side}>
<div className={styles.utbetalinger_side_innhold}>
<div className="flex flex-row justify-center gap-8 min-h-[40vh]">
{!isMobile && (
<Panel as="section" aria-label={t("filter.aria")} className={styles.filter_section}>
<Panel as="section" aria-label={t("filter.aria")} className="h-fit p-[1.5rem] pt-[1.5rem]">
<UtbetalingerFilter />
</Panel>
)}
<UtbetalingerPanelBeta />
<UtbetalingerPanel />
</div>
</div>
</FilterProvider>
Expand Down
71 changes: 0 additions & 71 deletions src/utbetalinger/Utbetalinger.test.ts

This file was deleted.

Loading
Loading