Skip to content

Commit

Permalink
Fjern stegUiid fra modellen
Browse files Browse the repository at this point in the history
  • Loading branch information
ssaegrov committed Feb 27, 2024
1 parent 9cd4d3c commit b061641
Show file tree
Hide file tree
Showing 11 changed files with 16 additions and 38 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ export function OppgaveStegMenyPunkt(behandlingSteg: IOppgaveSteg) {
return (
<li>
<NavLink
to={`${behandlingSteg.uuid}`}
to={`${behandlingSteg.stegNavn}`}
className={({ isActive }) =>
isActive ? classNames(styles.kulepunktActive, kulepunktKlasser) : kulepunktKlasser
}
Expand Down
1 change: 1 addition & 0 deletions app/entry.server.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ if (getEnv("USE_MSW") === "true") {
const server = setupMockServer();
startMockServer(server);
}

const csp = {
"img-src": ["'self'", "data:"],
"connect-src": ["'self'", "*.nav.no"], //trenger connect-src for å slenge faro metrics til nav sin oppsamler fra browser
Expand Down
6 changes: 2 additions & 4 deletions app/models/oppgave.server.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ export interface IOpplysning {
}

export interface IOppgaveSteg {
uuid: string;
stegNavn: string;
tilstand: IOppgaveStegTilstand;
opplysninger: IOpplysning[];
Expand Down Expand Up @@ -127,13 +126,12 @@ export async function avslagOppgave(
return { status: "success" };
}

export async function svarOppgaveSteg(
export async function svarOppgaveOpplysning(
oppgaveId: string,
stegId: string,
opplysninger: IOpplysning[],
session: SessionWithOboProvider,
): Promise<INetworkResponse> {
const url = `${getEnv("DP_SAKSBEHANDLING_URL")}/oppgave/${oppgaveId}/steg/${stegId}`;
const url = `${getEnv("DP_SAKSBEHANDLING_URL")}/oppgave/${oppgaveId}`;
const onBehalfOfToken = await getBehandlingOboToken(session);
const body = JSON.stringify(opplysninger);

Expand Down
6 changes: 2 additions & 4 deletions app/root.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import { json } from "@remix-run/node";
import type { MetaFunction } from "@remix-run/react";
import {
Links,
LiveReload,
Expand All @@ -17,7 +16,7 @@ import { cssBundleHref } from "@remix-run/css-bundle";
import globalCss from "~/global.css";
import { initFaro } from "~/utils/faro";

export const meta: MetaFunction = () => {
export function meta() {
return [
{
charset: "utf-8",
Expand All @@ -36,7 +35,7 @@ export const meta: MetaFunction = () => {
content: "Saksbehandlingløsning for dagpenger",
},
];
};
}

export function links() {
return [
Expand Down Expand Up @@ -98,7 +97,6 @@ export default function App() {
__html: `window.env = ${JSON.stringify(env)}`,
}}
/>
<Scripts />
<LiveReload />
</body>
</html>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import type { ActionFunctionArgs } from "@remix-run/node";
import { useActionData, useParams } from "@remix-run/react";
import invariant from "tiny-invariant";
import type { IOpplysning } from "~/models/oppgave.server";
import { svarOppgaveSteg } from "~/models/oppgave.server";
import { svarOppgaveOpplysning } from "~/models/oppgave.server";
import { OppgaveSteg } from "~/views/oppgave-steg/OppgaveSteg";
import { getSession } from "~/models/auth.server";
import { useTypedRouteLoaderData } from "~/hooks/useTypedRouteLoaderData";
Expand All @@ -18,13 +18,11 @@ import { isNetworkResponseError } from "~/utils/type-guards";
import { oppgaveErFerdigBehandlet } from "~/routes/saksbehandling.oppgave.$oppgaveId";

export async function action({ request, params }: ActionFunctionArgs) {
invariant(params.stegUuid, `params.stegUuid er påkrevd`);
invariant(params.oppgaveId, `params.oppgaveId er påkrevd`);

const session = await getSession(request);
const formData = await request.formData();
const skjemadata = parseSkjemadata<SkjemaMetadata>(formData, "metadata");
const stegUuid = params.stegUuid;

const validering = await hentValideringRegler(skjemadata.opplysninger).validate(formData);

Expand All @@ -33,7 +31,7 @@ export async function action({ request, params }: ActionFunctionArgs) {
return validationError(validering.error);
}

return await svarOppgaveSteg(params.oppgaveId, stegUuid, [], session);
return await svarOppgaveOpplysning(params.oppgaveId, [], session);
}

export interface SkjemaMetadata {
Expand All @@ -44,13 +42,13 @@ export default function OppgaveStegView() {
const { oppgave } = useTypedRouteLoaderData("routes/saksbehandling.oppgave.$oppgaveId");
const actionResponse = useActionData<typeof action>();

const { stegUuid } = useParams();
const steg = oppgave.steg.find((steg) => steg.uuid === stegUuid);
const { stegNavn } = useParams();
const steg = oppgave.steg.find((steg) => steg.stegNavn === stegNavn);

if (!steg) {
throw new Response(null, {
status: 500,
statusText: `Fant ikke steg med id: ${stegUuid}`,
statusText: `Fant ikke steg med navn: ${stegNavn}`,
});
}

Expand Down
2 changes: 1 addition & 1 deletion app/routes/saksbehandling.oppgave.$oppgaveId.steg.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ export default function OppgaveStegView() {
<div className={styles.oppgaveStegContainer}>
<ul>
{oppgave.steg.map((steg) => (
<OppgaveStegMenyPunkt key={steg.uuid} {...steg} />
<OppgaveStegMenyPunkt key={steg.stegNavn} {...steg} />
))}
</ul>

Expand Down
2 changes: 1 addition & 1 deletion app/routes/saksbehandling.oppgave.$oppgaveId.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ export async function loader({ params, request }: LoaderFunctionArgs) {

// dp-saksbehandling returnerer tomt array for steg frem til regelmotor har klargjort opplysninger som må bekreftes
if (oppgave.steg.length > 0 && !request.url.includes("/steg")) {
throw redirect(`steg/${oppgave.steg[0].uuid}`);
throw redirect(`steg/${oppgave.steg[0].stegNavn}`);
}

return json({ oppgave });
Expand Down
2 changes: 1 addition & 1 deletion app/routes/saksbehandling.oppgaver-behandlet.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ export default function SaksbehandlingOppgaverBehandlet() {
</Table.DataCell>
<Table.DataCell>
<RemixLink
to={`/saksbehandling/oppgave/${oppgaveId}/steg/${steg[0].uuid}`}
to={`/saksbehandling/oppgave/${oppgaveId}/steg/${steg[0].stegNavn}`}
asButtonVariant="primary"
>
Behandle
Expand Down
2 changes: 1 addition & 1 deletion app/views/oppgave-steg/OppgaveStegGenerell.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { ValidatedForm } from "remix-validated-form";
import { type SkjemaMetadata } from "~/routes/saksbehandling.oppgave.$oppgaveId.steg.$stegUuid";
import { type SkjemaMetadata } from "~/routes/saksbehandling.oppgave.$oppgaveId.steg.$stegNavn";
import { Button } from "@navikt/ds-react";
import { type IOppgaveStegProps } from "./OppgaveSteg";
import { useLocation, useNavigation } from "@remix-run/react";
Expand Down
17 changes: 0 additions & 17 deletions mocks/data/mock-oppgaver.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,61 +9,51 @@ export const masterMenyMock: IOppgave = {
journalpostIder: [],
steg: [
{
uuid: "msw-ikke-over-67",
stegNavn: "Ikke over 67 år",
tilstand: "Groenn",
opplysninger: [],
},
{
uuid: "msw-mangler-dokumentasjon",
stegNavn: "Mangler dokumentasjon",
tilstand: "Groenn",
opplysninger: [],
},
{
uuid: "msw-utdanning",
stegNavn: "Utdanning",
tilstand: "Groenn",
opplysninger: [],
},
{
uuid: "msw-reell-arbeidsoker",
stegNavn: "Reell arbeidsøker",
tilstand: "Groenn",
opplysninger: [],
},
{
uuid: "msw-ikke-utestengt",
stegNavn: "Ikke utestengt",
tilstand: "Groenn",
opplysninger: [],
},
{
uuid: "msw-lovvalg",
stegNavn: "Lovvalg",
tilstand: "Groenn",
opplysninger: [],
},
{
uuid: "msw-medlem",
stegNavn: "Medlem",
tilstand: "Groenn",
opplysninger: [],
},
{
uuid: "msw-opphold-i-norge",
stegNavn: "Opphold i Norge",
tilstand: "Groenn",
opplysninger: [],
},
{
uuid: "msw-arbeidsoker",
stegNavn: "Registrert som arbeidsøker",
tilstand: "Groenn",
opplysninger: [],
},
{
uuid: "msw-minste-arbeidsinntekt",
stegNavn: "Minste arbeidsinntekt",
tilstand: "Groenn",
opplysninger: [
Expand Down Expand Up @@ -98,19 +88,16 @@ export const masterMenyMock: IOppgave = {
],
},
{
uuid: "msw-tapt-arbeidsinntekt",
stegNavn: "Tapt arbeidsinntekt",
tilstand: "Groenn",
opplysninger: [],
},
{
uuid: "msw-tapt-arbeidstid",
stegNavn: "Tapt arbeidstid",
tilstand: "Groenn",
opplysninger: [],
},
{
uuid: "msw-ikke-fulle-folketrygdytelser",
stegNavn: "Ikke fulle folketrygdytelser",
tilstand: "Groenn",
opplysninger: [],
Expand Down Expand Up @@ -150,7 +137,6 @@ export const minsteinntektAvslagOppgave: IOppgave = {
journalpostIder: [],
steg: [
{
uuid: "sjalabaris",
stegNavn: "Gjenopptak / 8 uker",
tilstand: "Groenn",
opplysninger: [
Expand All @@ -169,7 +155,6 @@ export const minsteinntektAvslagOppgave: IOppgave = {
],
},
{
uuid: "962305f9-3ec5-482c-9d59-558c7653749a",
stegNavn: "Minste arbeidsinntekt",
tilstand: "Groenn",
opplysninger: [
Expand Down Expand Up @@ -247,7 +232,6 @@ export const ferdigBehandletMinsteinntektAvslagOppgave: IOppgave = {
journalpostIder: [],
steg: [
{
uuid: "msw-avsluttet-sjalabaris",
stegNavn: "Gjenopptak / 8 uker",
tilstand: "Groenn",
opplysninger: [
Expand All @@ -266,7 +250,6 @@ export const ferdigBehandletMinsteinntektAvslagOppgave: IOppgave = {
],
},
{
uuid: "msw-avsluttet-962305f9-3ec5-482c-9d59-558c7653749a",
stegNavn: "Minste arbeidsinntekt",
tilstand: "Groenn",
opplysninger: [
Expand Down
2 changes: 1 addition & 1 deletion tests/playwright/oppgaveliste.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,5 +19,5 @@ test("Man kan gå videre til en oppgave ", async ({ page, baseURL }) => {
.click();

await expect(page).toHaveURL(`
${baseURL}/oppgave/${oppgave.oppgaveId}/steg/${oppgave.steg[0].uuid}`);
${baseURL}/oppgave/${oppgave.oppgaveId}/steg/${oppgave.steg[0].stegNavn}`);
});

0 comments on commit b061641

Please sign in to comment.