diff --git a/frontend/README.md b/frontend/README.md index 593132a3..d3153d05 100644 --- a/frontend/README.md +++ b/frontend/README.md @@ -27,7 +27,8 @@ cp .env-example .env ## Sovelluksen ajaminen paikallisesti -Käynnistä sovellus kehitysmoodissa, jolloin se käyttää `mock`-hakemistossa olevaa mock-API:a ja Webpack lataa automaattisesti sovelluksen uudelleen selaimessa, kun teet muutoksia: +Käynnistä sovellus kehitysmoodissa, jolloin se käyttää `mock`-hakemistossa olevaa mock-API:a ja Webpack lataa +automaattisesti sovelluksen uudelleen selaimessa, kun teet muutoksia: ```shell npm run start:local @@ -41,7 +42,8 @@ npm run start:local:nologin Nyt voit mennä selaimella osoitteeseen http://localhost:8080/ . -Vaihtoehtoisesti voit tehdä lokaalin dev-buildin (`npm run build:local`) ja tarjoilla `dist`-hakemistossa olevat tiedostot jollain toisella HTTP-palvelinsovelluksella. +Vaihtoehtoisesti voit tehdä lokaalin dev-buildin (`npm run build:local`) ja tarjoilla `dist`-hakemistossa olevat +tiedostot jollain toisella HTTP-palvelinsovelluksella. Saat [Oppija-raamit](https://github.com/Opetushallitus/oppija-raamit) käyttöön `PROXY_OPPIJA_RAAMIT` -ympäristömuuttujan avulla seuraavasti: @@ -62,7 +64,9 @@ Jest-yksikkötestit: npm run unit ``` -Testissä on käytetty [Jestin snapshoteja](https://jestjs.io/docs/en/snapshot-testing). Jos teet toiminnallisen muutoksen, joka aiheuttaa olemassaolevan snapshotin muuttumisen, voit päivittää snapshotin komennolla `npm run unit -- -u`, mutta ole tarkkana ettet vahingossa tallenna rikkinäisiä snapshoteja versionhallintaan. +Testissä on käytetty [Jestin snapshoteja](https://jestjs.io/docs/en/snapshot-testing). Jos teet toiminnallisen muutoksen, +joka aiheuttaa olemassaolevan snapshotin muuttumisen, voit päivittää snapshotin komennolla `npm run unit -- -u`, mutta +ole tarkkana ettet vahingossa tallenna rikkinäisiä snapshoteja versionhallintaan. ## Koodityylit diff --git a/frontend/jest.config.js b/frontend/jest.config.js index 686b6a0b..c1c36627 100644 --- a/frontend/jest.config.js +++ b/frontend/jest.config.js @@ -9,8 +9,7 @@ module.exports = { ], moduleFileExtensions: ['js', 'jsx', 'json'], moduleNameMapper: { - '\\.(svg)$': '/src/__mocks__/filemock.js', - 'Resources/mapping/usagePermissionDescriptions': '/resources/mapping/usagePermissionDescriptions.json' + '\\.(svg)$': '/src/__mocks__/filemock.js' }, snapshotSerializers: ['enzyme-to-json/serializer'], coverageThreshold: { diff --git a/frontend/mock/api.json b/frontend/mock/api.json index d9ede3a4..7b36e69d 100644 --- a/frontend/mock/api.json +++ b/frontend/mock/api.json @@ -26,7 +26,16 @@ } ], "timestamps": ["2018-09-19T12:05:26.432+03", "2018-09-20T16:01:44.543+03"], - "serviceName": "koski" + "serviceName": "koski", + "isMyDataUse": true, + "isJakolinkkiUse": false + }, + { + "organizations": [], + "timestamps": ["2023-09-16T12:05:26.432+03", "2023-09-20T16:01:44.543+03"], + "serviceName": "koski", + "isMyDataUse": true, + "isJakolinkkiUse": true }, { "organizations": [ @@ -89,7 +98,9 @@ "2018-07-11T14:44:14.543+03", "2018-06-27T00:47:38.321+03" ], - "serviceName": "koski" + "serviceName": "koski", + "isMyDataUse": false, + "isJakolinkkiUse": false }, { "organizations": [ diff --git a/frontend/mock/mapping/usagePermissionDescriptions.json b/frontend/mock/mapping/usagePermissionDescriptions.json deleted file mode 100644 index 1345bce8..00000000 --- a/frontend/mock/mapping/usagePermissionDescriptions.json +++ /dev/null @@ -1,4 +0,0 @@ -{ - "1.1.111.111.11.11111111111": "Olet antanut tälle palvelutarjoajalle luvan käyttää tietojasi.", - "default": "Lakiin perustuva tiedonkäyttölupa." -} diff --git a/frontend/package-lock.json b/frontend/package-lock.json index 018a1ea3..83c69a46 100644 --- a/frontend/package-lock.json +++ b/frontend/package-lock.json @@ -5176,9 +5176,9 @@ "integrity": "sha1-FkpUg+Yw+kMh5a8HAg5TGDGyYJs=" }, "node_modules/caniuse-lite": { - "version": "1.0.30001577", - "resolved": "https://registry.npmjs.org/caniuse-lite/-/caniuse-lite-1.0.30001577.tgz", - "integrity": "sha512-rs2ZygrG1PNXMfmncM0B5H1hndY5ZCC9b5TkFaVNfZ+AUlyqcMyVIQtc3fsezi0NUCk5XZfDf9WS6WxMxnfdrg==", + "version": "1.0.30001690", + "resolved": "https://registry.npmjs.org/caniuse-lite/-/caniuse-lite-1.0.30001690.tgz", + "integrity": "sha512-5ExiE3qQN6oF8Clf8ifIDcMRCRE/dMGcETG/XGMD8/XiXm6HXQgQTh1yZYLXXpSOsEUlJm1Xr7kGULZTuGtP/w==", "funding": [ { "type": "opencollective", @@ -5192,7 +5192,8 @@ "type": "github", "url": "https://github.com/sponsors/ai" } - ] + ], + "license": "CC-BY-4.0" }, "node_modules/chalk": { "version": "2.4.2", @@ -19860,9 +19861,9 @@ "integrity": "sha1-FkpUg+Yw+kMh5a8HAg5TGDGyYJs=" }, "caniuse-lite": { - "version": "1.0.30001577", - "resolved": "https://registry.npmjs.org/caniuse-lite/-/caniuse-lite-1.0.30001577.tgz", - "integrity": "sha512-rs2ZygrG1PNXMfmncM0B5H1hndY5ZCC9b5TkFaVNfZ+AUlyqcMyVIQtc3fsezi0NUCk5XZfDf9WS6WxMxnfdrg==" + "version": "1.0.30001690", + "resolved": "https://registry.npmjs.org/caniuse-lite/-/caniuse-lite-1.0.30001690.tgz", + "integrity": "sha512-5ExiE3qQN6oF8Clf8ifIDcMRCRE/dMGcETG/XGMD8/XiXm6HXQgQTh1yZYLXXpSOsEUlJm1Xr7kGULZTuGtP/w==" }, "chalk": { "version": "2.4.2", diff --git a/frontend/resources/mapping/translations.json b/frontend/resources/mapping/translations.json index 4e9699c8..a6770ff0 100644 --- a/frontend/resources/mapping/translations.json +++ b/frontend/resources/mapping/translations.json @@ -7,10 +7,15 @@ "sv": "På den här sidan kan du granska och administrera de tillstånd du har givit om dina uppgifter. Du ser här dessutom vilka aktörer, t.ex. myndigheter, som har använt dina uppgifter.", "en": "On this page you can inspect and manage permissions you have given to use your data. You can also check which organisations (e.g. authorities) have used your data." }, - "Tietoja käyttäneissä toimijoissa on listattuna kaikki ne kerrat, kun tietojasi on katsottu tai niitä on haettu toiseen järjestelmään automatisoidusti. Tietojasi voivat hakea automaattisesti Opetushallituksen eri järjestelmät, viranomaiset tai ne kolmannet osapuolet, joille olet erikseen antanut luvan hyödyntää tietojasi Oma Opintopolusta. Opetushallituksen ja viranomaisten automaattiset haut tapahtuvat yleensä ruuhka-aikojen ulkopuolella, lähinnä ilta- tai yöaikaan.": { - "fi": "Tietoja käyttäneissä toimijoissa on listattuna kaikki ne kerrat, kun tietojasi on katsottu tai niitä on haettu toiseen järjestelmään automatisoidusti. Tietojasi voivat hakea automaattisesti Opetushallituksen eri järjestelmät, viranomaiset tai ne kolmannet osapuolet, joille olet erikseen antanut luvan hyödyntää tietojasi Oma Opintopolusta. Opetushallituksen ja viranomaisten automaattiset haut tapahtuvat yleensä ruuhka-aikojen ulkopuolella, lähinnä ilta- tai yöaikaan.", - "sv": "I aktörer som använt uppgifter är listade alla de gånger då dina uppgifter har granskats eller hämtats automatiskt till ett annat system. Dina uppgifter kan automatiskt hämtas av olika system från Utbildningsstyrelsen, myndigheter eller de tredje parter som du specifikt har gett tillstånd att använda dina uppgifter från Min Studieinfo. Utbildningsstyrelsens och myndigheternas automatiska sökningar sker vanligtvis utanför rusningstider, främst på kvällar eller nätter.", - "en": "Organizations that have used your data includes each occasion when your data has been viewed or automatically retrieved to another system. Your data can be automatically retrieved by various systems from the Finnish National Agency for Education, authorities, or third parties to whom you’ve specifically granted permission to use your data from My Studyinfo. Automatic searches by the Finnish National Agency for Education and authorities typically occur outside peak hours, primarily in the evenings or at night." + "Tietoja käyttäneissä toimijoissa on listattuna kaikki ne kerrat, kun ...": { + "fi": "Tietoja käyttäneissä toimijoissa on listattuna niitä kertoja, kun tietojasi on katsottu tai niitä on haettu toiseen järjestelmään automatisoidusti. Tietojasi voivat hakea automaattisesti Opetushallituksen eri järjestelmät, viranomaiset tai ne kolmannet osapuolet, joille olet erikseen antanut luvan hyödyntää tietojasi Oma Opintopolusta. Opetushallituksen järjestelmien ja viranomaisten automaattiset haut tapahtuvat yleensä ruuhka-aikojen ulkopuolella, lähinnä ilta- tai yöaikaan. Joidenkin Opetushallituksen ylläpitämien järjestelmien opintotietojen haut näkyvät vain kyseisen järjestelmän lokitiedoissa, eivätkä listaudu Tietojani käyttäneet toimijat -näkymään. Tilastoinnissa tai erilaisissa tiedonsiirtojen tarkistusraporteissa ja massahakutoiminnallisuuksissa käytettyjen tietojen käyttökertoja ei ole koostettuna Tietojani käyttäneet toimijat -näkymään.", + "sv": "TOOO: Tietoja käyttäneissä toimijoissa on listattuna niitä kertoja, kun tietojasi on katsottu tai niitä on haettu toiseen järjestelmään automatisoidusti. Tietojasi voivat hakea automaattisesti Opetushallituksen eri järjestelmät, viranomaiset tai ne kolmannet osapuolet, joille olet erikseen antanut luvan hyödyntää tietojasi Oma Opintopolusta. Opetushallituksen järjestelmien ja viranomaisten automaattiset haut tapahtuvat yleensä ruuhka-aikojen ulkopuolella, lähinnä ilta- tai yöaikaan. Joidenkin Opetushallituksen ylläpitämien järjestelmien opintotietojen haut näkyvät vain kyseisen järjestelmän lokitiedoissa, eivätkä listaudu Tietojani käyttäneet toimijat -näkymään. Tilastoinnissa tai erilaisissa tiedonsiirtojen tarkistusraporteissa ja massahakutoiminnallisuuksissa käytettyjen tietojen käyttökertoja ei ole koostettuna Tietojani käyttäneet toimijat -näkymään.", + "en": "TODO: Tietoja käyttäneissä toimijoissa on listattuna niitä kertoja, kun tietojasi on katsottu tai niitä on haettu toiseen järjestelmään automatisoidusti. Tietojasi voivat hakea automaattisesti Opetushallituksen eri järjestelmät, viranomaiset tai ne kolmannet osapuolet, joille olet erikseen antanut luvan hyödyntää tietojasi Oma Opintopolusta. Opetushallituksen järjestelmien ja viranomaisten automaattiset haut tapahtuvat yleensä ruuhka-aikojen ulkopuolella, lähinnä ilta- tai yöaikaan. Joidenkin Opetushallituksen ylläpitämien järjestelmien opintotietojen haut näkyvät vain kyseisen järjestelmän lokitiedoissa, eivätkä listaudu Tietojani käyttäneet toimijat -näkymään. Tilastoinnissa tai erilaisissa tiedonsiirtojen tarkistusraporteissa ja massahakutoiminnallisuuksissa käytettyjen tietojen käyttökertoja ei ole koostettuna Tietojani käyttäneet toimijat -näkymään." + }, + "Tietojani käyttäneet toimijat -näkymässä ovat listattuna myös ...": { + "fi": "Tietojani käyttäneet toimijat -näkymässä ovat listattuna myös omien opintosuoritusten jakolinkkien käyttökerrat. Tämä tarkoittaa tilannetta, kun olet luonut Oma Opintopolku -palvelussa jakolinkin omista opintosuorituksistasi ja jakanut sen eteenpäin haluamallesi kolmannelle osapuolelle. Tietojani käyttäneet toimijat -näkymässä ei ole kuitenkaan saatavilla tietoa, kuka tai mikä taho linkin on avannut. Ainoastaan linkin katselu- tai käyttökerrat ovat nähtävissä palvelussa.", + "sv": "TODO: Tietojani käyttäneet toimijat -näkymässä ovat listattuna myös omien opintosuoritusten jakolinkkien käyttökerrat. Tämä tarkoittaa tilannetta, kun olet luonut Oma Opintopolku -palvelussa jakolinkin omista opintosuorituksistasi ja jakanut sen eteenpäin haluamallesi kolmannelle osapuolelle. Tietojani käyttäneet toimijat -näkymässä ei ole kuitenkaan saatavilla tietoa, kuka tai mikä taho linkin on avannut. Ainoastaan linkin katselu- tai käyttökerrat ovat nähtävissä palvelussa.", + "en": "TODO: Tietojani käyttäneet toimijat -näkymässä ovat listattuna myös omien opintosuoritusten jakolinkkien käyttökerrat. Tämä tarkoittaa tilannetta, kun olet luonut Oma Opintopolku -palvelussa jakolinkin omista opintosuorituksistasi ja jakanut sen eteenpäin haluamallesi kolmannelle osapuolelle. Tietojani käyttäneet toimijat -näkymässä ei ole kuitenkaan saatavilla tietoa, kuka tai mikä taho linkin on avannut. Ainoastaan linkin katselu- tai käyttökerrat ovat nähtävissä palvelussa." }, "Taulukko tietojen käyttökerroista": { "fi": "Näytä tietojen käyttökerrat", @@ -156,5 +161,15 @@ "fi": "Tarkasteltava henkilö:", "sv": "Person vars uppgifter granskas", "en": "Person who’s data is being viewed" + }, + "Opintosuorituksista tehdyn jakolinkin tuntematon käyttäjä": { + "fi": "Opintosuorituksista tehdyn jakolinkin tuntematon käyttäjä", + "sv": "TODO: Opintosuorituksista tehdyn jakolinkin tuntematon käyttäjä", + "en": "TODO: Opintosuorituksista tehdyn jakolinkin tuntematon käyttäjä" + }, + "Oma opintopolku -palvelussa luomiesi omien opintosuoritusten jakolinkkien käyttökerrat": { + "fi": "Oma opintopolku -palvelussa luomiesi omien opintosuoritusten jakolinkkien käyttökerrat", + "sv": "TODO: Oma opintopolku -palvelussa luomiesi omien opintosuoritusten jakolinkkien käyttökerrat", + "en": "TODO: Oma opintopolku -palvelussa luomiesi omien opintosuoritusten jakolinkkien käyttökerrat" } } diff --git a/frontend/resources/mapping/usagePermissionDescriptions.json b/frontend/resources/mapping/usagePermissionDescriptions.json deleted file mode 100644 index cdddfc6b..00000000 --- a/frontend/resources/mapping/usagePermissionDescriptions.json +++ /dev/null @@ -1,5 +0,0 @@ -{ - "1.2.246.562.10.77876988401": "Olet antanut tälle palvelutarjoajalle luvan käyttää tietojasi.", - "1.2.246.562.10.46399742280": "Olet antanut tälle palvelutarjoajalle luvan käyttää tietojasi.", - "default": "Lakiin perustuva tiedonkäyttölupa." -} diff --git a/frontend/src/component/Header.jsx b/frontend/src/component/Header.jsx index 90d8eeac..0705e998 100644 --- a/frontend/src/component/Header.jsx +++ b/frontend/src/component/Header.jsx @@ -18,11 +18,15 @@ const Header = ({ selectedHetu, onSelectHetu }) => (

{t`Tällä sivulla voit tarkastella ja hallinnoida antamiasi käyttölupia tietoihisi. Lisäksi näet mitkä toimijat, esim. viranomaiset, ovat käyttäneet tietojasi.`}{'*'}

-

{t`Tietoja käyttäneissä toimijoissa on listattuna kaikki ne kerrat, kun tietojasi on katsottu tai niitä on haettu toiseen järjestelmään automatisoidusti. Tietojasi voivat hakea automaattisesti Opetushallituksen eri järjestelmät, viranomaiset tai ne kolmannet osapuolet, joille olet erikseen antanut luvan hyödyntää tietojasi Oma Opintopolusta. Opetushallituksen ja viranomaisten automaattiset haut tapahtuvat yleensä ruuhka-aikojen ulkopuolella, lähinnä ilta- tai yöaikaan.`}

+

{t`Tietoja käyttäneissä toimijoissa on listattuna kaikki ne kerrat, kun ...`}

+ { + /* TODO: Toistaiseksi anonyymit suoritusjakolinkkien katselut eivät näy, koska ne puuttuvat DynamoDB:stä */ + /*

{t`Tietojani käyttäneet toimijat -näkymässä ovat listattuna myös ...`}

*/ + } - + ) Header.propTypes = ({ diff --git a/frontend/src/component/__snapshots__/Header.test.jsx.snap b/frontend/src/component/__snapshots__/Header.test.jsx.snap index d0ce5023..5ed8a8a6 100644 --- a/frontend/src/component/__snapshots__/Header.test.jsx.snap +++ b/frontend/src/component/__snapshots__/Header.test.jsx.snap @@ -15,7 +15,7 @@ exports[`Header should be rendered 1`] = ` *

- Tietoja käyttäneissä toimijoissa on listattuna kaikki ne kerrat, kun tietojasi on katsottu tai niitä on haettu toiseen järjestelmään automatisoidusti. Tietojasi voivat hakea automaattisesti Opetushallituksen eri järjestelmät, viranomaiset tai ne kolmannet osapuolet, joille olet erikseen antanut luvan hyödyntää tietojasi Oma Opintopolusta. Opetushallituksen ja viranomaisten automaattiset haut tapahtuvat yleensä ruuhka-aikojen ulkopuolella, lähinnä ilta- tai yöaikaan. + Tietoja käyttäneissä toimijoissa on listattuna niitä kertoja, kun tietojasi on katsottu tai niitä on haettu toiseen järjestelmään automatisoidusti. Tietojasi voivat hakea automaattisesti Opetushallituksen eri järjestelmät, viranomaiset tai ne kolmannet osapuolet, joille olet erikseen antanut luvan hyödyntää tietojasi Oma Opintopolusta. Opetushallituksen järjestelmien ja viranomaisten automaattiset haut tapahtuvat yleensä ruuhka-aikojen ulkopuolella, lähinnä ilta- tai yöaikaan. Joidenkin Opetushallituksen ylläpitämien järjestelmien opintotietojen haut näkyvät vain kyseisen järjestelmän lokitiedoissa, eivätkä listaudu Tietojani käyttäneet toimijat -näkymään. Tilastoinnissa tai erilaisissa tiedonsiirtojen tarkistusraporteissa ja massahakutoiminnallisuuksissa käytettyjen tietojen käyttökertoja ei ole koostettuna Tietojani käyttäneet toimijat -näkymään.

view(nameLens, organizations) -const Organization = ({ organizationAlternatives, timestamps, serviceName }) => ( - - +const Organization = ({ organizationAlternatives, timestamps, serviceName, isMyDataUse, isJakolinkkiUse }) => ( + + ) @@ -23,7 +23,9 @@ const Organization = ({ organizationAlternatives, timestamps, serviceName }) => Organization.propTypes = { organizationAlternatives: PropTypes.array.isRequired, timestamps: PropTypes.array.isRequired, - serviceName: PropTypes.string.isRequired + serviceName: PropTypes.string.isRequired, + isMyDataUse: PropTypes.bool, + isJakolinkkiUse: PropTypes.bool } export default Organization diff --git a/frontend/src/component/organization/OrganizationDetails.jsx b/frontend/src/component/organization/OrganizationDetails.jsx index bb8ccafc..1b62aefb 100644 --- a/frontend/src/component/organization/OrganizationDetails.jsx +++ b/frontend/src/component/organization/OrganizationDetails.jsx @@ -2,7 +2,7 @@ import React from 'react' import PropTypes from 'prop-types' import styled from 'styled-components' import t from 'util/translate' -import { getTranslatedUsagePermissionDescription, isMydataPartner } from 'util/usagePermissionDescriptions' +import { getTranslatedUsagePermissionDescription } from 'util/usagePermissionDescriptions' import { Bold } from 'ui/typography' import constants from 'ui/constants' import media from 'ui/media' @@ -31,27 +31,28 @@ const Description = styled.div` font-size: ${constants.font.size.s}; ` -const weblink = oid => (isMydataPartner(oid) +const weblink = isMyDataUse => (isMyDataUse ? t('omadata-link') : t('tietosuojaseloste-link') ) -const OrganizationDetails = ({ organizationOid }) => ( +const OrganizationDetails = ({ isMyDataUse, isJakolinkkiUse }) => (
- {t`Tietojen käyttölupa`}: {getTranslatedUsagePermissionDescription(organizationOid)} + {t`Tietojen käyttölupa`}: {getTranslatedUsagePermissionDescription(isMyDataUse, isJakolinkkiUse)}
) OrganizationDetails.propTypes = { - organizationOid: PropTypes.string.isRequired + isMyDataUse: PropTypes.bool, + isJakolinkkiUse: PropTypes.bool } export default OrganizationDetails diff --git a/frontend/src/component/organization/Organizations.jsx b/frontend/src/component/organization/Organizations.jsx index 1cbd9a11..676ec49d 100644 --- a/frontend/src/component/organization/Organizations.jsx +++ b/frontend/src/component/organization/Organizations.jsx @@ -11,7 +11,7 @@ const Organizations = ({ translatedOrganizations }) => ( { - translatedOrganizations.map(({ organizations, timestamps, serviceName }) => { + translatedOrganizations.map(({ organizations, timestamps, serviceName, isMyDataUse, isJakolinkkiUse }) => { const key = map(view(oidLens), organizations).join(',') return ( @@ -20,6 +20,8 @@ const Organizations = ({ translatedOrganizations }) => ( organizationAlternatives={organizations} timestamps={timestamps} serviceName={serviceName} + isMyDataUse={isMyDataUse} + isJakolinkkiUse={isJakolinkkiUse} /> ) }) diff --git a/frontend/src/component/organization/__snapshots__/Organization.test.jsx.snap b/frontend/src/component/organization/__snapshots__/Organization.test.jsx.snap index f0f3748c..a3d31bab 100644 --- a/frontend/src/component/organization/__snapshots__/Organization.test.jsx.snap +++ b/frontend/src/component/organization/__snapshots__/Organization.test.jsx.snap @@ -153,7 +153,7 @@ exports[`Organization should render with organization permitted by user, and exp > Tietojen käyttölupa: - Olet antanut tälle palvelutarjoajalle luvan käyttää tietojasi. + Lakiin perustuva tiedonkäyttölupa. { - const permission = permissionsMap[organizationOid] || permissionsMap.default +export const getTranslatedUsagePermissionDescription = (isMyDataUse, isJakolinkkiUse) => { + const permission = (isMyDataUse && isJakolinkkiUse) + ? 'Oma opintopolku -palvelussa luomiesi omien opintosuoritusten jakolinkkien käyttökerrat' + : isMyDataUse + ? 'Olet antanut tälle palvelutarjoajalle luvan käyttää tietojasi.' + : 'Lakiin perustuva tiedonkäyttölupa.' return t(permission) } - -export const isMydataPartner = organizationOid => { - return Boolean(permissionsMap[organizationOid]) -}