Skip to content

Commit

Permalink
OK-671: Muutettu laskettuihin valinnanvaiheisiin liittyvien tyyppien …
Browse files Browse the repository at this point in the history
…nimeämistä

- Käytetty "Model"-päätettä tyypeille, jotka kuvaa rajapinnasta tulevaa dataa
  • Loading branch information
pretseli committed Nov 14, 2024
1 parent d1b9473 commit f3b5018
Show file tree
Hide file tree
Showing 7 changed files with 29 additions and 32 deletions.
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use client';
import { useTranslations } from '@/app/hooks/useTranslations';
import { LaskettuValinnanVaihe } from '@/app/lib/types/laskenta-types';
import { LaskettuValinnanVaiheModel } from '@/app/lib/types/laskenta-types';
import { toFormattedDateTimeString } from '@/app/lib/localization/translation-utils';
import { LaskettuJonoWithHakijaInfo } from '@/app/hooks/useLasketutValinnanVaiheet';
import { getValintatapaJonoNimi } from '@/app/lib/get-valintatapa-jono-nimi';
Expand All @@ -10,7 +10,7 @@ export const ValintatapajonoAccordionTitle = ({
valinnanVaihe,
jono,
}: {
valinnanVaihe: LaskettuValinnanVaihe;
valinnanVaihe: LaskettuValinnanVaiheModel;
jono: LaskettuJonoWithHakijaInfo;
}) => {
const { t } = useTranslations();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use client';
import { Box } from '@mui/material';
import { LaskettuValinnanVaihe } from '@/app/lib/types/laskenta-types';
import { LaskettuValinnanVaiheModel } from '@/app/lib/types/laskenta-types';
import { AccordionBox } from '@/app/components/accordion-box';
import { useJonosijatSearch } from '@/app/hooks/useJonosijatSearch';
import { TablePaginationWrapper } from '@/app/components/table/table-pagination-wrapper';
Expand Down Expand Up @@ -75,7 +75,7 @@ export const ValintatapajonoContent = ({
jono,
}: {
hakukohdeOid: string;
valinnanVaihe: Omit<LaskettuValinnanVaihe, 'valintatapajonot'>;
valinnanVaihe: Omit<LaskettuValinnanVaiheModel, 'valintatapajonot'>;
jono: LaskettuJonoWithHakijaInfo;
}) => {
const { t } = useTranslations();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import useToaster from '@/app/hooks/useToaster';
import { LaskettuValinnanVaihe } from '@/app/lib/types/laskenta-types';
import { LaskettuValinnanVaiheModel } from '@/app/lib/types/laskenta-types';
import {
muutaSijoittelunStatus,
MuutaSijoittelunStatusProps,
Expand All @@ -15,7 +15,7 @@ export const useSijoitteluStatusMutation = (hakukohdeOid: string) => {
await muutaSijoittelunStatus({ jono, status });
queryClient.setQueryData(
['getLasketutValinnanVaiheet', hakukohdeOid],
(vaiheet: Array<LaskettuValinnanVaihe>) =>
(vaiheet: Array<LaskettuValinnanVaiheModel>) =>
vaiheet.map((vaihe) => ({
...vaihe,
valintatapajonot: vaihe.valintatapajonot?.map((oldJono) => ({
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ import {
import { isEmpty, isNonNullish } from 'remeda';
import { useState } from 'react';
import { ChevronRight } from '@mui/icons-material';
import { LasketutValinnanvaiheetInternal } from '@/app/hooks/useLasketutValinnanVaiheet';
import { LasketutValinnanvaiheet } from '@/app/hooks/useLasketutValinnanVaiheet';
import { toFormattedDateTimeString } from '@/app/lib/localization/translation-utils';
import { getValintatapaJonoNimi } from '@/app/lib/get-valintatapa-jono-nimi';
import { SijoitteluajonTulosHakutoive } from '@/app/lib/valinta-tulos-service';
Expand All @@ -34,7 +34,7 @@ import { TFunction } from 'i18next';

type Tulokset = {
sijoittelunTulokset: SijoitteluajonTulosHakutoive;
valinnanvaiheet?: LasketutValinnanvaiheetInternal;
valinnanvaiheet?: LasketutValinnanvaiheet;
};

type HakukohdeTuloksilla = Hakukohde & Tulokset;
Expand Down
29 changes: 13 additions & 16 deletions src/app/hooks/useLasketutValinnanVaiheet.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,15 +4,15 @@ import { getHakukohteenLasketutValinnanvaiheet } from '../lib/valintalaskenta-se
import { getHakemukset } from '../lib/ataru';
import { TranslatedName } from '../lib/localization/localization-types';
import {
JonoSija,
LaskettuValinnanVaihe,
LaskettuValintatapajono,
JonoSijaModel,
LaskettuValinnanVaiheModel,
LaskettuValintatapajonoModel,
} from '../lib/types/laskenta-types';
import { indexBy, map, mapKeys, omit, pipe, prop, sortBy } from 'remeda';
import { HakemuksenTila } from '@/app/lib/types/ataru-types';

export type JonoSijaInternal = Omit<
JonoSija,
export type JonoSija = Omit<
JonoSijaModel,
'jarjestyskriteerit' | 'harkinnanvarainen' | 'prioriteetti'
> & {
pisteet?: number;
Expand All @@ -26,25 +26,22 @@ type AdditionalHakemusFields = {
hakemuksenTila: HakemuksenTila;
};

export type JonoSijaWithHakijaInfo = JonoSijaInternal & AdditionalHakemusFields;
export type JonoSijaWithHakijaInfo = JonoSija & AdditionalHakemusFields;

export type LaskettuJonoInternal = Omit<
LaskettuValintatapajono,
'jonosijat'
> & {
jonosijat: Array<JonoSijaInternal>;
export type LaskettuJono = Omit<LaskettuValintatapajonoModel, 'jonosijat'> & {
jonosijat: Array<JonoSija>;
};

export type LaskettuJonoWithHakijaInfo = Omit<
LaskettuValintatapajono,
LaskettuValintatapajonoModel,
'jonosijat'
> & {
jonosijat: Array<JonoSijaWithHakijaInfo>;
};

export type LasketutValinnanvaiheetInternal = Array<
Omit<LaskettuValinnanVaihe, 'valintatapajonot'> & {
valintatapajonot?: Array<LaskettuJonoInternal>;
export type LasketutValinnanvaiheet = Array<
Omit<LaskettuValinnanVaiheModel, 'valintatapajonot'> & {
valintatapajonot?: Array<LaskettuJono>;
}
>;

Expand All @@ -54,7 +51,7 @@ export const selectValinnanvaiheet = <
lasketutValinnanvaiheet,
selectHakemusFields,
}: {
lasketutValinnanvaiheet?: Array<LaskettuValinnanVaihe>;
lasketutValinnanvaiheet?: Array<LaskettuValinnanVaiheModel>;
selectHakemusFields?: (hakemusOid: string) => H;
}) => {
return pipe(
Expand Down
10 changes: 5 additions & 5 deletions src/app/lib/types/laskenta-types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ export type Jarjestyskriteeri = {
};
};

export type JonoSija = {
export type JonoSijaModel = {
jonosija: number;
hakemusOid: string;
hakijaOid: string;
Expand All @@ -46,23 +46,23 @@ export type JonoSija = {
jarjestyskriteerit: Array<Jarjestyskriteeri>;
};

export type LaskettuValintatapajono = {
export type LaskettuValintatapajonoModel = {
oid: string;
nimi: string;
valintatapajonooid: string;
prioriteetti: number;
jonosijat: Array<JonoSija>;
jonosijat: Array<JonoSijaModel>;
valmisSijoiteltavaksi: boolean;
siirretaanSijoitteluun: boolean;
};

export type LaskettuValinnanVaihe = {
export type LaskettuValinnanVaiheModel = {
jarjestysnumero: number;
valinnanvaiheoid: string;
hakuOid: string;
nimi: string;
createdAt: number;
valintatapajonot?: Array<LaskettuValintatapajono>;
valintatapajonot?: Array<LaskettuValintatapajonoModel>;
};

export type SeurantaTiedot = {
Expand Down
6 changes: 3 additions & 3 deletions src/app/lib/valintalaskenta-service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import {
HakijaryhmanHakija,
HakukohteenHakijaryhma,
JarjestyskriteeriTila,
LaskettuValinnanVaihe,
LaskettuValinnanVaiheModel,
SeurantaTiedot,
} from './types/laskenta-types';
import {
Expand Down Expand Up @@ -39,7 +39,7 @@ import {
export const getHakukohteenLasketutValinnanvaiheet = async (
hakukohdeOid: string,
) => {
const response = await client.get<Array<LaskettuValinnanVaihe>>(
const response = await client.get<Array<LaskettuValinnanVaiheModel>>(
configuration.hakukohteenLasketutValinnanVaiheetUrl({ hakukohdeOid }),
);
return response.data;
Expand All @@ -54,7 +54,7 @@ export type HakemuksenValintalaskentaData = {
oid: string;
prioriteetti: number;
hakukohdeRyhmaOids: Array<string>;
valinnanvaihe: Array<LaskettuValinnanVaihe>;
valinnanvaihe: Array<LaskettuValinnanVaiheModel>;
kaikkiJonotSijoiteltu: boolean;
harkinnanvaraisuus: boolean;
hakijaryhma: Array<unknown>;
Expand Down

0 comments on commit f3b5018

Please sign in to comment.