Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

YKI(Frontend) OPHYKIKEH-301: change title when changing route #581

Merged
merged 6 commits into from
Oct 24, 2023

Conversation

jrkkp
Copy link
Contributor

@jrkkp jrkkp commented Oct 3, 2023

Yhteenveto

Titlen muutos kun route vaihtuu. Toteutus kopioitu VKT:stä.

@@ -3,7 +3,7 @@
"common": {
"actions": "Toiminnot",
"appNameAbbreviation": "YKI",
"appTitle": "Yleiset kielitutkinnot | Opetushallitus",
"appTitle": "Yleiset kielitutkinnot - Opetushallitus",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Saman muutoksen voisi tehdä myös ruotsinkielisiin käännöksiin.

},
"pageTitle": {
"registration": "Ilmoittautuminen",
"initRegistration": "Ilmoittaudu yleiseen kielitutkintoon (YKI)",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tämä Ilmoittaudu yleiseen kielitutkintoon (YKI) taitaa olla kopioitu käännösavaimesta yki.component.registration.header, jota ei itse asiassa taideta enää käyttää. Tuon käyttämättömän käännösavaimen voisi poistaa ja samalla tämän otsikon voisi muuttaa.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tässä voisi mahdollisesti otsikkona toimia sama kuin mikä näkyy stepperin alla headingina: Tunnistaudu ilmoittautumista varten. Joku muukin vaihtoehto voisi toimia.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Voisi ehkä myös olla pelkkä Ilmoittautuminen jos haluttaisiin pitää sama otsikko koko ilmoittautumisflow'n ajan.

"pageTitle": {
"registration": "Ilmoittautuminen",
"initRegistration": "Ilmoittaudu yleiseen kielitutkintoon (YKI)",
"examDetails": "Tutkinto",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Olisiko tässäkin vain Ilmoittautuminen? Tämä on siis käytännössä sen ilmoittautumislomakkeen sivu, mutta sivulle voi tulla myös suoralla URLilla jolloin on mahdollista että ilmoittautuminen on täynnä tai jostain muusta syystä lomaketta ei pääse täyttämään.

"registration": "Ilmoittautuminen",
"initRegistration": "Ilmoittaudu yleiseen kielitutkintoon (YKI)",
"examDetails": "Tutkinto",
"registrationPaymentStatus": "Ilmoittaudu yleiseen kielitutkintoon (YKI)",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tämä käännösavain liittyy sivuun joka näytetään Paytrailista palatessa, jolloin maksu voi olla joko hyväksytty, keskeytetty (käyttäjän toimesta) tai muuten epäonnistunut. Ehkä edelleen vain Ilmoittautuminen, jolloin sama otsikko pysyisi läpi koko ilmoittautumisflow'n?

"registrationPaymentStatus": "Ilmoittaudu yleiseen kielitutkintoon (YKI)",
"reassessment": "Tarkistusarviointi",
"evaluationOrder": "Pyydä tarkistusarviointia",
"evaluationOrderStatus": "Pyydä tarkistusarviointia",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ehkä vain Tarkistusarviointi? Näytetään taas Paytrailista palatessa, jolloin kyseessä voi olla onnistunut tai epäonnistunut maksu.

Copy link
Contributor

@pkoivisto pkoivisto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hyvältä näyttää.

@jrkkp jrkkp merged commit 02d68d7 into dev Oct 24, 2023
@jrkkp jrkkp deleted the feature/OPHYKIKEH-301 branch October 24, 2023 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants