Tampereen Seutuselvitysraportti osa 1: kuukausittaisten arvojen CSV #6179
Conversation
tmkrepo
changed the title
tmkrepo
force-pushed
the
seutuselvitys-report
branch
from
a873524 to
fd3999a
Compare
tmkrepo
force-pushed
the
seutuselvitys-report
branch
from
fd3999a to
15af447
Compare
| @@ -20,6 +20,7 @@ class EspooActionRuleMapping : ActionRuleMapping { | |||
| Action.Global.SUBMIT_PATU_REPORT -> sequenceOf(HasGlobalRole(UserRole.ADMIN)) | |||
| Action.Global.READ_NON_SSN_CHILDREN_REPORT -> | |||
| sequenceOf(HasGlobalRole(UserRole.ADMIN, UserRole.SERVICE_WORKER)) | |||
| Action.Global.READ_TAMPERE_REGIONAL_SURVEY_REPORT -> sequenceOf() | |||
There was a problem hiding this comment.
Tässä voisi käyttää myös
| Action.Global.READ_TAMPERE_REGIONAL_SURVEY_REPORT -> sequenceOf() | |
| Action.Global.READ_TAMPERE_REGIONAL_SURVEY_REPORT -> emptySequence() |
There was a problem hiding this comment.
Siistitty esitettyyn muotoon.
| ) | ||
|
|
||
| data class MonthlyMunicipalDaycareResult( | ||
| val month: Int, |
There was a problem hiding this comment.
Näistä yksittäisistä *Result luokista voisi poistaa tuon kuukausitiedon, koska se on jo tuossa RegionalSurveyMonthlyResults.month päätasolla saatavilla
There was a problem hiding this comment.
Tyyppikohtaiset Result-luokat poistettu kokonaan ja kaikki arvot kasattu samaan objektiin jo bäkkärin puolella.
| ...mc.municipalDaycareResults, | ||
| ...mc.familyDaycareResults, | ||
| ...mc.municipalShiftCareResults, | ||
| ...mc.assistanceResults |
There was a problem hiding this comment.
Tässä voisi käyttää ylemmän tason month tietoa
| ...mc.assistanceResults | |
| ...mc.assistanceResults, | |
| month: mc.month |
There was a problem hiding this comment.
Kuukausitiedon toisteisuus ja destrukturoinnin tarve poistunut palautettavan data classin refaktoroinnin myötä.
| ...mc.municipalShiftCareResults, | ||
| ...mc.assistanceResults | ||
| })), | ||
| [(i) => i.month] |
There was a problem hiding this comment.
Tämän voisi tehdä myös näin:
| [(i) => i.month] | |
| 'month', | |
| 'asc' |
There was a problem hiding this comment.
Tästä käytiin toisen PR:n yhteydessä keskustelua, ja siellä funktio-malli nousi etusijalle, sillä se pakottaa ottamaan kantaa järjestyskriteeriin tyypityksen läpi -> helppojen virheiden tekeminen vaikeutuu.
| } | ||
| } | ||
|
|
||
| const sortedReportResult = useMemo( |
There was a problem hiding this comment.
Olisiko tämä eri resulttien yhdistely parempi tehdä jo API päässä, jos tuota jaotellumpaa datamuotoa ei käytetä frontissa?
There was a problem hiding this comment.
Rajapinnan palauttama kuukausitulosten data class on nyt refaktoroitu niin, että se sisältää suoraan kuukauden ja kaikki yksittäiset tilastoarvot. Tämä ratkaisee myös kuukausitiedon toisteisuuden, koska sisin tyypitysrakenne poistuu.
Luo uuden raportin, jolla on tarkoitus mahdollistaa Tampereen seudulla vuosittain tehtävän Seutuselvityksen tietojen kerääminen eVakan raportointia käyttäen.
Raportti on näkyvissä ytimen pääkäyttäjä-roolilla, mutta piilotettu kokonaan Espoon käyttöoikeusasetuksissa.
Tässä toteutettuna raportin perusrakenne ja kuukausittaisten arvojen CSV. Raporttiin on tulossa myös muita CSV-tiedostoja.
Raportti tulee näkyville automaattisesti, mutta on käyttökelpoinen vain Tampereen seudun konfiguraatioilla, joten se kannattaa piilottaa muualla ylikirjoittamalla käyttöoikeus seuraavasti:
Action.Global.READ_TAMPERE_REGIONAL_SURVEY_REPORT -> emptySequence()Dokumentaatio