HAI-1864 Remove hanke size from hanke form (#368)

Size of hanke is no longer needed, so removed hanke size checkboxes and size summary from hanke form.
City-of-Helsinki · Sep 13, 2023 · 0793950 · 0793950
1 parent b076b4f
commit 0793950
Show file tree

Hide file tree

Showing 13 changed files with 17 additions and 91 deletions.
diff --git a/src/domain/hanke/edit/HankeForm.styles.scss b/src/domain/hanke/edit/HankeForm.styles.scss
@@ -1,8 +1,6 @@
 @import 'src/assets/styles/layout.scss';
 
 .hankeForm {
-  padding: 0 var(--spacing-l) 0 var(--spacing-l);
-
   &__formWpr {
     display: flex;
   }

diff --git a/src/domain/hanke/edit/HankeForm.test.tsx b/src/domain/hanke/edit/HankeForm.test.tsx
@@ -68,7 +68,7 @@ describe('HankeForm', () => {
         onFormClose={handleFormClose}
       >
         child
-      </HankeForm>
+      </HankeForm>,
     );
 
     fireEvent.change(screen.getByTestId(FORMFIELD.NIMI), { target: { value: nimi } });
@@ -97,7 +97,7 @@ describe('HankeForm', () => {
         onFormClose={() => ({})}
       >
         child
-      </HankeForm>
+      </HankeForm>,
     );
     expect(screen.getByTestId(FORMFIELD.NIMI)).toHaveValue('Formin nimi');
     expect(screen.getByTestId(FORMFIELD.KUVAUS)).toHaveValue('Formin kuvaus');

diff --git a/src/domain/hanke/edit/HankeFormPerustiedot.tsx b/src/domain/hanke/edit/HankeFormPerustiedot.tsx
@@ -6,12 +6,7 @@ import { useFormContext } from 'react-hook-form';
 import { Box } from '@chakra-ui/react';
 import Dropdown from '../../../common/components/dropdown/Dropdown';
 import TextInput from '../../../common/components/textInput/TextInput';
-import {
-  HANKE_VAIHE,
-  HANKE_SUUNNITTELUVAIHE,
-  HANKE_TYOMAATYYPPI,
-  HANKE_TYOMAAKOKO,
-} from '../../types/hanke';
+import { HANKE_VAIHE, HANKE_SUUNNITTELUVAIHE, HANKE_TYOMAATYYPPI } from '../../types/hanke';
 import { FORMFIELD, FormProps } from './types';
 import { useFormPage } from './hooks/useFormPage';
 import DropdownMultiselect from '../../../common/components/dropdown/DropdownMultiselect';
@@ -32,8 +27,6 @@ const HankeFormPerustiedot: React.FC<React.PropsWithChildren<FormProps>> = ({
 
   // Subscribe to vaihe changes
   const hankeVaiheField = watch(FORMFIELD.VAIHE);
-  // Subscribe to tyomaaKoko changes
-  const tyomaaKokoField = watch(FORMFIELD.TYOMAAKOKO);
 
   useEffect(() => {
     if (hankeVaiheField !== HANKE_VAIHE.SUUNNITTELU) {
@@ -120,42 +113,19 @@ const HankeFormPerustiedot: React.FC<React.PropsWithChildren<FormProps>> = ({
           disabled={hankeVaiheField !== HANKE_VAIHE.SUUNNITTELU}
         />
       </div>
-      <div className="form3">
-        <div className="dataWpr">
-          <div className="formWpr">
-            <DropdownMultiselect
-              name={FORMFIELD.TYOMAATYYPPI}
-              id={FORMFIELD.TYOMAATYYPPI}
-              options={$enum(HANKE_TYOMAATYYPPI).map((value) => ({
-                value,
-                label: t(`hanke:${FORMFIELD.TYOMAATYYPPI}:${value}`),
-              }))}
-              defaultValue={formData ? (formData[FORMFIELD.TYOMAATYYPPI] as string[]) : []}
-              label={t(`hankeForm:labels:${FORMFIELD.TYOMAATYYPPI}`)}
-              invalid={!!errors[FORMFIELD.TYOMAATYYPPI]}
-              errorMsg={t('hankeForm:insertFieldError')}
-            />
-          </div>
-          <div className="formWpr">
-            <SelectionGroup
-              direction="vertical"
-              label={t(`hankeForm:labels:${FORMFIELD.TYOMAAKOKO}`)}
-            >
-              {$enum(HANKE_TYOMAAKOKO).map((value) => {
-                return (
-                  <RadioButton
-                    {...register(FORMFIELD.TYOMAAKOKO)}
-                    label={t(`hanke:${FORMFIELD.TYOMAAKOKO}:${value}`)}
-                    key={value}
-                    id={value}
-                    value={value}
-                    checked={tyomaaKokoField === value}
-                  />
-                );
-              })}
-            </SelectionGroup>
-          </div>
-        </div>
+      <div className="formWpr">
+        <DropdownMultiselect
+          name={FORMFIELD.TYOMAATYYPPI}
+          id={FORMFIELD.TYOMAATYYPPI}
+          options={$enum(HANKE_TYOMAATYYPPI).map((value) => ({
+            value,
+            label: t(`hanke:${FORMFIELD.TYOMAATYYPPI}:${value}`),
+          }))}
+          defaultValue={formData ? (formData[FORMFIELD.TYOMAATYYPPI] as string[]) : []}
+          label={t(`hankeForm:labels:${FORMFIELD.TYOMAATYYPPI}`)}
+          invalid={!!errors[FORMFIELD.TYOMAATYYPPI]}
+          errorMsg={t('hankeForm:insertFieldError')}
+        />
       </div>
       <div className="formWpr">
         <br />

diff --git a/src/domain/hanke/edit/components/BasicInformationSummary.tsx b/src/domain/hanke/edit/components/BasicInformationSummary.tsx
@@ -67,10 +67,6 @@ const BasicInformationSummary: React.FC<Props> = ({ formData, children }) => {
           {formData.tyomaaTyyppi?.map((tyyppi) => t(`hanke:tyomaaTyyppi:${tyyppi}`)).join(', ')}
         </p>
       </SectionItemContent>
-      <SectionItemTitle>{t('hankeForm:labels:tyomaaKoko')}</SectionItemTitle>
-      <SectionItemContent>
-        <p>{formData.tyomaaKoko && <>{t(`hanke:tyomaaKoko:${formData.tyomaaKoko}`)}</>}</p>
-      </SectionItemContent>
       <SectionItemTitle>{t('hankeForm:labels:onYKTHanke')}</SectionItemTitle>
       <SectionItemContent>
         <p>{formData.onYKTHanke ? t('common:yes') : t('common:no')}</p>

diff --git a/src/domain/hanke/edit/types.ts b/src/domain/hanke/edit/types.ts
@@ -14,7 +14,6 @@ export enum FORMFIELD {
   KATUOSOITE = 'tyomaaKatuosoite',
   SUUNNITTELUVAIHE = 'suunnitteluVaihe',
   TYOMAATYYPPI = 'tyomaaTyyppi',
-  TYOMAAKOKO = 'tyomaaKoko',
   HAITTA_ALKU_PVM = 'haittaAlkuPvm',
   HAITTA_LOPPU_PVM = 'haittaLoppuPvm',
   KAISTAHAITTA = 'kaistaHaitta',

diff --git a/src/domain/hanke/hankeView/HankeView.test.tsx b/src/domain/hanke/hankeView/HankeView.test.tsx
@@ -52,9 +52,6 @@ test('Correct information about hanke should be displayed', async () => {
   expect(screen.queryByText('24.2.2023')).toBeInTheDocument();
   expect(screen.queryByText('Ohjelmointi')).toBeInTheDocument();
   expect(screen.queryByText('Kaukolämpö')).toBeInTheDocument();
-  expect(
-    screen.queryByText('Työmaa ulottuu kadun eri puolille ja/tai usean korttelin mittainen työmaa'),
-  ).toBeInTheDocument();
   expect(screen.queryByText('Ei')).toBeInTheDocument();
   expect(screen.queryByText('11974 m²')).toBeInTheDocument();
 

diff --git a/src/domain/mocks/data/hankkeet-data.ts b/src/domain/mocks/data/hankkeet-data.ts
@@ -1,4 +1,4 @@
-import { HankeDataDraft, HANKE_POLYHAITTA, HANKE_TYOMAAKOKO } from '../../types/hanke';
+import { HankeDataDraft, HANKE_POLYHAITTA } from '../../types/hanke';
 
 const hankkeet: HankeDataDraft[] = [
   {
@@ -36,7 +36,6 @@ const hankkeet: HankeDataDraft[] = [
     loppuPvm: '2024-11-27T00:00:00Z',
     vaihe: 'OHJELMOINTI',
     suunnitteluVaihe: null,
-    tyomaaKoko: HANKE_TYOMAAKOKO.SUPPEA_TAI_PISTE,
     version: 0,
     createdBy: '1',
     createdAt: '2022-11-27T11:43:43.481215Z',
@@ -250,7 +249,6 @@ const hankkeet: HankeDataDraft[] = [
     vaihe: 'OHJELMOINTI',
     suunnitteluVaihe: null,
     tyomaaTyyppi: ['KAUKOLAMPO'],
-    tyomaaKoko: 'LAAJA_TAI_USEA_KORTTELI',
     version: 0,
     createdBy: '1',
     createdAt: '2022-11-27T11:44:22.443735Z',

diff --git a/src/domain/mocks/data/hankkeet.ts b/src/domain/mocks/data/hankkeet.ts
@@ -25,7 +25,6 @@ export async function create(data: HankeDataDraft) {
     suunnitteluVaihe: null,
     tyomaaKatuosoite: '',
     tyomaaTyyppi: [],
-    tyomaaKoko: null,
     alueet: [],
     ...data,
   };

diff --git a/src/domain/mocks/hankeList.ts b/src/domain/mocks/hankeList.ts
@@ -67,7 +67,6 @@ const hankeList: HankeData[] = [
       },
     ],
     tyomaaKatuosoite: '',
-    tyomaaKoko: null,
     tormaystarkasteluTulos: null,
     liikennehaittaindeksi: null,
   },
@@ -137,7 +136,6 @@ const hankeList: HankeData[] = [
       },
     ],
     tyomaaKatuosoite: '',
-    tyomaaKoko: null,
     tormaystarkasteluTulos: null,
     liikennehaittaindeksi: null,
   },

diff --git a/src/domain/types/hanke.ts b/src/domain/types/hanke.ts
@@ -52,13 +52,6 @@ export enum HANKE_TYOMAATYYPPI {
 }
 export type HANKE_TYOMAATYYPPI_KEY = keyof typeof HANKE_TYOMAATYYPPI;
 
-export enum HANKE_TYOMAAKOKO {
-  SUPPEA_TAI_PISTE = 'SUPPEA_TAI_PISTE',
-  YLI_10M_TAI_KORTTELI = 'YLI_10M_TAI_KORTTELI',
-  LAAJA_TAI_USEA_KORTTELI = 'LAAJA_TAI_USEA_KORTTELI',
-}
-export type HANKE_TYOMAAKOKO_KEY = keyof typeof HANKE_TYOMAAKOKO;
-
 export enum HANKE_KAISTAHAITTA {
   YKSI = 'YKSI',
   KAKSI = 'KAKSI',
@@ -205,7 +198,6 @@ export interface HankeData {
   suunnitteluVaihe: HANKE_SUUNNITTELUVAIHE_KEY | null;
   tyomaaKatuosoite: string | null;
   tyomaaTyyppi: HANKE_TYOMAATYYPPI_KEY[];
-  tyomaaKoko: HANKE_TYOMAAKOKO_KEY | null;
   alueet: HankeAlue[];
   liikennehaittaindeksi: LiikenneHaittaIndeksi | null;
   omistajat?: Array<HankeContact>;

diff --git a/src/locales/en.json b/src/locales/en.json
@@ -351,11 +351,6 @@
       "RAKENNUS_TAI_TOTEUTUS": "Construction planning or implementation planning",
       "TYOMAAN_TAI_HANKKEEN_AIKAINEN": "Early planning of a worksite or project"
     },
-    "tyomaaKoko": {
-      "SUPPEA_TAI_PISTE": "Single, narrow location and/or point-like location",
-      "YLI_10M_TAI_KORTTELI": "Worksite length of more than 10 m and/or worksite the length of an entire block",
-      "LAAJA_TAI_USEA_KORTTELI": "Worksite extends to covering different sides of the street and/or is several blocks long"
-    },
     "tyomaaTyyppi": {
       "VESI": "Water",
       "VIEMARI": "Sewer",
@@ -465,7 +460,6 @@
       "alkuPvm": "Project start date",
       "loppuPvm": "Project end date",
       "tyomaaTyyppi": "Worksite type",
-      "tyomaaKoko": "Worksite size",
       "haittaAlkuPvm": "Start date of the nuisance",
       "haittaLoppuPvm": "End date of the nuisance",
       "kaistaHaitta": "Lane hindrance",
@@ -769,7 +763,6 @@
       "katuosoite": "Street address",
       "avaaKartalla": "Open on a map",
       "hankkeenKesto": "Project duration",
-      "tyomaanKoko": "Worksite size",
       "tyomaanTyyppi": "Worksite type",
       "hankkeenHaitat": "Project nuisances",
       "haitanKesto": "Duration of the nuisance",

diff --git a/src/locales/fi.json b/src/locales/fi.json
@@ -360,11 +360,6 @@
       "RAKENNUS_TAI_TOTEUTUS": "Rakennus- tai toteutussuunnittelu",
       "TYOMAAN_TAI_HANKKEEN_AIKAINEN": "Työmaan tai hankkeen aikainen suunnittelu"
     },
-    "tyomaaKoko": {
-      "SUPPEA_TAI_PISTE": "Yksittäinen suppea kohde ja/tai pistemäinen työmaa",
-      "YLI_10M_TAI_KORTTELI": "Työmaa-alueen pituus yli 10 m ja/tai korttelin mittainen työmaa",
-      "LAAJA_TAI_USEA_KORTTELI": "Työmaa ulottuu kadun eri puolille ja/tai usean korttelin mittainen työmaa"
-    },
     "tyomaaTyyppi": {
       "VESI": "Vesi",
       "VIEMARI": "Viemäri",
@@ -474,7 +469,6 @@
       "alkuPvm": "Hankkeen alkupäivä",
       "loppuPvm": "Hankkeen loppupäivä",
       "tyomaaTyyppi": "Työn tyyppi",
-      "tyomaaKoko": "Hankkeen koko",
       "haittaAlkuPvm": "Haittojen alkupäivä",
       "haittaLoppuPvm": "Haittojen loppupäivä",
       "kaistaHaitta": "Autoliikenteen kaistahaitta",
@@ -778,7 +772,6 @@
       "katuosoite": "Katuosoite",
       "avaaKartalla": "Avaa kartalla",
       "hankkeenKesto": "Hankkeen kesto",
-      "tyomaanKoko": "Työmaan koko",
       "tyomaanTyyppi": "Työmaantyyppi",
       "hankkeenHaitat": "Hankkeen haitat",
       "haitanKesto": "Haitan kesto",

diff --git a/src/locales/sv.json b/src/locales/sv.json
@@ -351,11 +351,6 @@
       "RAKENNUS_TAI_TOTEUTUS": "Bygg- eller genomförandeplanering",
       "TYOMAAN_TAI_HANKKEEN_AIKAINEN": "Planering under bygg- eller projekttid"
     },
-    "tyomaaKoko": {
-      "SUPPEA_TAI_PISTE": "Enstaka trång plats eller punktformig byggarbetsplats",
-      "YLI_10M_TAI_KORTTELI": "Byggarbetsplatsområdets längd över 10 m och/eller byggarbetsplats i storlek som kvarter",
-      "LAAJA_TAI_USEA_KORTTELI": "Byggplatsen sträcker sig till båda sidor av gatan och/eller över flera kvarter"
-    },
     "tyomaaTyyppi": {
       "VESI": "Vatten",
       "VIEMARI": "Avlopp",
@@ -465,7 +460,6 @@
       "alkuPvm": "Projektets startdatum",
       "loppuPvm": "Projektets slutdatum",
       "tyomaaTyyppi": "Typ av byggarbetsplats",
-      "tyomaaKoko": "Byggarbetsplatsens storlek",
       "haittaAlkuPvm": "Startdatum för olägenhet",
       "haittaLoppuPvm": "Slutdatum för olägenhet",
       "kaistaHaitta": "Filolägenhet",
@@ -769,7 +763,6 @@
       "katuosoite": "Gatuadress",
       "avaaKartalla": "Öppna på karta",
       "hankkeenKesto": "Projektets varaktighet",
-      "tyomaanKoko": "Byggarbetsplatsens storlek",
       "tyomaanTyyppi": "Typ av byggarbetsplats",
       "hankkeenHaitat": "Projektets olägenheter",
       "haitanKesto": "Olägenhetens varaktighet",