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

OK-747 #70

Merged
merged 12 commits into from
Feb 10, 2025
Merged

OK-747 #70

merged 12 commits into from
Feb 10, 2025

Conversation

pretseli
Copy link
Contributor

@pretseli pretseli commented Feb 7, 2025

Oletko lisännyt tarvittavat yksikkö- tai ui-testit toiminnallisuudelle? Kyllä
Oletko tarkistanut ja päivittänyt riippuvuudet? En
Oletko kokeillut toimiiko käyttöliittymä mobiilissa landscape moodissa? Kyllä
Oletko testannut että lisäämäsi toiminto on saavutettava? Kyllä

…valinnanvaiheille

Siirretty ValintalaskentaEditModal sekä sen käyttämät komponentit app/components-hakemistoon.
…n muokkauksen jälkeen

- Henkilöittäin näkymässä noudetaan eri rajapinnalla kuin hakukohteen valintalaskennan-tulokset-välilehdellä. Annetaan onSuccess-parametri, jonka avulla noudetaan uudelleen käyttäen oikeaa rajapintaa kummassakin tapauksessa.
@pretseli pretseli marked this pull request as ready for review February 10, 2025 07:35
Copy link
Contributor

@SalamaGofore SalamaGofore left a comment

Choose a reason for hiding this comment

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

Näyttää hyvältä, mutta laitoin parit kommentit

tests/e2e/valintalaskennan-tulokset.spec.ts Outdated Show resolved Hide resolved
src/app/components/valintalaskenta-edit-modal.tsx Outdated Show resolved Hide resolved
@pretseli pretseli merged commit 0b76e3a into main Feb 10, 2025
5 checks passed
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