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.

Sign up for GitHub

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

Initial version of localization #12

Merged
merged 11 commits into from
May 23, 2024
Merged

Initial version of localization #12

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
37dd506
Initial version of localization
SalamaGofore May 16, 2024
4bea529
refactor localization
SalamaGofore May 16, 2024
02002bb
Move fetching translations to api route
SalamaGofore May 16, 2024
b2f1c17
Rename test
SalamaGofore May 16, 2024
7d46879
Fix hakukohde translation
SalamaGofore May 17, 2024
33d919a
Localize ui-elements
SalamaGofore May 20, 2024
959f2c2
Keyword rest of the strings to localize
SalamaGofore May 20, 2024
36a6abc
Use local files in development
SalamaGofore May 20, 2024
664d6b2
Couple fixes and change tila enum
SalamaGofore May 20, 2024
fa8b80d
Add playwright test for lokalisaatio and update README
SalamaGofore May 21, 2024
f76b854
Use useTranslations hook and use only finnish in translation keys
SalamaGofore May 23, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 34 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,40 @@

Valintojen toteuttamisen käyttöliittymä

## Lokaaliajo

Asenna riippuvuudet komennolla:

`npm ci`

Sen jälkeen käynnistä palvelu komennolla:

`npm run dev`

## Testaus

Aja yksikkötestit komennolla:

`npm test`
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
`npm test`
`npm run test`

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ei tarvi, npm test toimii

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ä on varmaan ainoa npm-skriptin ajava komento, jossa ei ole pakko laittaa run. Yhdenmukaisuus on hyve tässäkin, vaikka node ei olisikaan yhdenmukainen.


### Kälitestit

Käynnistä sovelluskomennolla:

`npm run dev-test`

Aja sen jälkeen testit komennolla:

`npx playwright test`

Jos haluat ajaa testit vain tietyllä selaimella niin se onnistuu komennolla:

`npx playwright test --project=firefox`

Jos haluat ajaa vain tietyn testitiedoston, se onnistuu komennolla:

`npx playwright test --project=chromium tests/e2e/lokalisointi.spec.ts`

## Deploy

Asenna ensin sovelluksen riippuvuudet ja buildaa next.js sovellus:
Expand Down
Loading
Loading