AE-1977: Show link for hallinto-oikeus attachment for käskypäätös / varsinainen päätös toimenpide #976
Conversation
Juholei
changed the title
Juholei
requested review from
muep,
solita-antti-mottonen and
solita-juhohaa
…arsinainen päätös toimenpide
Juholei
force-pushed
the
feature/AE-1977
branch
from
d40c536 to
3d3be93
Compare
| henkilo: { | ||
| pdf: valvontaApi.url.documentHenkilo, | ||
| courtAttachment: valvontaApi.url.courtAttachmentHenkilo | ||
| }, | ||
| yritys: { | ||
| pdf: valvontaApi.url.documentYritys, | ||
| courtAttachment: valvontaApi.url.courtAttachmentYritys | ||
| } |
There was a problem hiding this comment.
Tämä nyt on vain tällainen mietintä, mutta olisiko kivempaa, jos tässä pääsisi kutsumaan noita attachmenttejä näin
courtAttachment: R.partial(valvontaApi.url.attachmentYritys, ["hallinto-oikeus.pdf"])
There was a problem hiding this comment.
Mietin jotain vastaavaa itsekin aluksi, mutta ajattelin sitten mennä vain yksinkertaisemmalla toteutuksella, kun muiden liitteiden olemassaolo on toistaiseksi ainoastaan teoreettinen mahdollisuus.
| courtAttachmentHenkilo: (henkiloId, id, toimenpideId) => | ||
| `${url.toimenpide( | ||
| id, | ||
| toimenpideId | ||
| )}/henkilot/${henkiloId}/attachment/hallinto-oikeus.pdf`, | ||
| courtAttachmentYritys: (yritysId, id, toimenpideId) => | ||
| `${url.toimenpide( | ||
| id, | ||
| toimenpideId | ||
| )}/yritykset/${yritysId}/attachment/hallinto-oikeus.pdf`, |
There was a problem hiding this comment.
Tämä on tuota edellistä mietintää:
Tässä olisi siis courtAttachmentYritys sijaan attachmentYritys, joka ottaisi myös attachmentName
There was a problem hiding this comment.
Olisi kyllä varmaan vähän premature optimization, eikä ehkä myöskään "Keep it simple" 🤔
There was a problem hiding this comment.
Mutta siis ehdottomasti näin sitten, jos meille tulee muita liitteitä.
…äätös toimenpidetype
…arsinainen päätös toimenpide