OK-670: Henkilöittäin haku ja suodatus #51
Conversation
- Henkilösivupalkki haulla (vain nimestä haku toistaiseksi) - Listauksessa nimi ja hetu - Sivupalkista klikkailemalla navigoidaan yksinkertaiseen henkilö-näkymään
… erilliseen komponenttiin
…rilliseksi komponentiksi
pretseli
force-pushed
the
OK-670-henkiloittain-suodatus
branch
from
c68c804 to
ba8dcb9
Compare
| <InfoItem label={t('henkilo.hakemus-oid')} value={hakemus.hakemusOid} /> | ||
| <InfoItem | ||
| label={t('henkilo.lahiosoite')} | ||
| value={`${hakemus.lahiosoite}, ${hakemus.postinumero}`} |
There was a problem hiding this comment.
Sisältääkö tuo postinumero myös postitoimipaikan?
There was a problem hiding this comment.
Ei sisällä. Lisäsin tuohon vasta helpoimmat jutut, koska henkilöittäin-näkymän sisällön näyttämiselle on eri tiketti, jonka teen tämän jälkeen.
| @@ -90,5 +90,13 @@ export const THEME_OVERRIDES: ThemeOptions = { | |||
| }), | |||
| }, | |||
| }, | |||
| MuiFormHelperText: { | |||
There was a problem hiding this comment.
Jos tää on yleisempi vois jossain kohti laittaa design system -kirjastoon?
There was a problem hiding this comment.
Kyllä, siinä vaiheessa kun sinne lisätään jokin komponentti jossa tätä käytetään.
|
|
||
| const MINIMIZED_PANEL_WIDTH = '60px'; | ||
|
|
||
| const StyledPanel = styled('aside')({ |
There was a problem hiding this comment.
Vastaavaan ois valmis Muin Drawer-komponentti mut sinänsä tämäkin ajaa asian.
Nyt paljastan pihallaoloni mutta mikäs palikka tuo 'aside' on?
There was a problem hiding this comment.
Lonkalta heitän, että on aika paljon työläämpää yrittää vääntää Mui:n Drawer näyttämään ja toimimaan silleen kuin tässä halutaan.
Aside on vain semanttinen HTML-elementti: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/aside
Nyt kun tarkemmin lueskelin, niin ei ehkä sittenkään ole aivan oikea elementti tähän. Ehkä vaihdan tuon jossain vaiheessa vain div:ksi.
| @@ -5,6 +5,53 @@ import VALINTAKOKEET from '@tests/e2e/fixtures/valintakokeet.json'; | |||
| import VALINTAKOEOSALLISTUMISET from '@tests/e2e/fixtures/valintakoeosallistumiset.json'; | |||
| import HAKEMUKSET from '@tests/e2e/fixtures/hakeneet.json'; | |||
|
|
|||
| const HAKEMUKSET_BY_OID = { | |||
There was a problem hiding this comment.
👍 kiva että vähennetty testidatan copypastea
tests/e2e/henkilo.spec.ts
Outdated
| await Promise.all([ | ||
| henkiloSearchInput.fill('123456-123X'), | ||
| page.waitForRequest((request) => | ||
| request.url().includes('henkilotunnus=123456-123X'), |
There was a problem hiding this comment.
Tässä on nyt assertoitu että se hakuehto tulee searchParamseihin mutta ei sitä että listaus suodattuu sen perusteella (toki apin palauttama asia pitää sitten mockata)
Oletko lisännyt tarvittavat yksikkö- tai ui-testit toiminnallisuudelle? Kyllä
Oletko tarkistanut ja päivittänyt riippuvuudet? En
Oletko kokeillut toimiiko käyttöliittymä mobiilissa landscape moodissa? Kyllä
Oletko testannut että lisäämäsi toiminto on saavutettava? Kyllä